Added content to the Sponsorship/Prospectus Page#38
Added content to the Sponsorship/Prospectus Page#38Hellrungj wants to merge 3 commits intoundergroundjs:masterfrom
Conversation
>> - Added Limited to Row >> - Added A La Carte Sponsorship Options Table >> - Added Limited to row >> - Added Branding in conference media >> - Added Branding on t-shirt >> - Added Conference ticket
kwelch
left a comment
kwelch
left a comment
There was a problem hiding this comment.
Thank you so much for working on this! I left some comments with some thoughts I had. Much appreciated.
src/pages/become-a-sponsor.js
Outdated
| ∞ | ||
| </span> | ||
| </td> | ||
| </tr> |
There was a problem hiding this comment.
I think I would rather have them reach out or we cross of the row when sold out than to display the counts.
There was a problem hiding this comment.
I'm going to remove this row then. And we can add a call to action elsewhere.
src/pages/become-a-sponsor.js
Outdated
| <tr> | ||
| <th>Benefit</th> | ||
| <th css={styles.highlight}>Platinum (Limit 1)</th> | ||
| <th css={styles.highlight}>Platinum</th> |
There was a problem hiding this comment.
I think leaving limit one here may be better than listing limits on all of them.
src/pages/become-a-sponsor.js
Outdated
| </tbody> | ||
| </table> | ||
|
|
||
| <Styled.h3>A La Carte Sponsorship Options</Styled.h3> |
There was a problem hiding this comment.
This section replaces the Exclusive Sponsorship options, so we should probably remove it: https://github.com/undergroundjs/website/pull/38/files#diff-c0a411ccae889ddca60b183f7439007fR90-R100
src/pages/become-a-sponsor.js
Outdated
| </span> | ||
| </td> | ||
| <td> | ||
| <span role="img" aria-label="Benefit included"> |
There was a problem hiding this comment.
the aria-label is only needed to wrap the emoji so those using a screenreader know what the check means. We should remove it in the cases where it is wrapping a number because it will override the number and it will lose meaning.
Added Limit 1 to the Platinum Cal Heading Removed A La Carte Sponsorship Option Table Update the numbers in the table to not include image span
|
Updated the Sponsorship/Prospectus Page. |
|
I am good with this as is for now. Can we add a link to this in the footer. |
3 participants
Update prospectus for 2020 #35 Update prospectus for 2020