Skip to content

Conversation

Bros112
Copy link
Contributor

@Bros112 Bros112 commented Sep 8, 2025

I added reference to the free t-shirt perk and the Discord coloured roles on the membership page.
This fixes issue #839 and enhances the clarity of the signup benefits for new potential members.

Added a mention of the free tshirt perk.
This fixes issue CSSUoB#839


Signed-off-by: Bros112 <[email protected]>
Added coloured heart emojis to represent the choices people have.


Signed-off-by: Bros112 <[email protected]>
@Bros112 Bros112 requested a review from a team as a code owner September 8, 2025 01:26
@cssbhamdev cssbhamdev enabled auto-merge (squash) September 8, 2025 01:26
@MattyTheHacker MattyTheHacker added enhancement sync Keep this pr up to date with it's base branch automatically labels Sep 8, 2025
@MattyTheHacker MattyTheHacker linked an issue Sep 8, 2025 that may be closed by this pull request
@MattyTheHacker
Copy link
Member

looks like one of the emoji's hasn't worked
image

otherwise looks good

@Bros112
Copy link
Contributor Author

Bros112 commented Sep 8, 2025

On my windows computer it's fine, but on my android phone, its not. I think it's a problem with supported emoji. I included the pink heart to be accurate to the colours of roles in the server (merging both green ones together). A fix for this could be to not include the pink heart, due to the lack of support across all platforms. This would, however, reduce the accuracy of the colour indicators.

The pink heart emoji is not supported on all platforms.
Therefore I am removing it, so there are no visual -
inconsistencies between platforms.

Signed-off-by: Bros112 <[email protected]>
auto-merge was automatically disabled September 8, 2025 13:54

Head branch was pushed to by a user without write access

@MattyTheHacker MattyTheHacker enabled auto-merge (squash) September 9, 2025 00:06
@MattyTheHacker MattyTheHacker enabled auto-merge (squash) September 9, 2025 00:07
@MattyTheHacker MattyTheHacker merged commit 1a0393a into CSSUoB:main Sep 9, 2025
11 checks passed
@Bros112 Bros112 deleted the patch-1 branch September 9, 2025 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement sync Keep this pr up to date with it's base branch automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Free T-shirt perk
2 participants