Implement improved "selected" component

Description

 


See the new design

(include but don’t update the icon as this will happen globally, later)

 

Environment

None

Activity

Show:
Duminda Aluthgamage
2 days ago

Latest version live

Duminda Aluthgamage
May 1, 2021, 12:34 AM

This was completed and updated in tst - no changes were made to the typography in this ticket, however some other updates are being tracked in

Duminda Aluthgamage
May 1, 2021, 12:32 AM

so this has to be marked completed as it was completed by

Then, the ticket I just closed needs to be reopened as it becomes a design ticket to make new updates to the component.

If/when need to open another dev ticket to make updates, we can create a new one for the queue

Duminda Aluthgamage
May 1, 2021, 12:16 AM

Apologies, missed this one , and wasn’t tracking this because I had incorrectly placed it in “Entity Page Redesign” instead of “Entity Finder Redesign”

However, I agree that we will not make any changes to the existing color palette until we have finalized the Synapse color palette overall

I agree with the choice of #1A1C29 over #515359 for accessibility

#323131 is not in our design files, its the default for synapse.org today

Stockard Simon
April 21, 2021, 5:57 PM

I’m going to have to disagree on this one. But before I go into details about this, please note that visual styles are currently being worked on and all of these colors could change in the upcoming weeks.

The color being proposed, #515359 is not AAA accessible when used on different color backgrounds (for instance, any notifications). This is why the reworked first round of designs ( ) proposed a darker color for text.

I’m not sure where #323131 is coming from, I’m not seeing that color in any of my files or in the Synapse palette but between those 2 I strongly suggest we go forward with that color to be accessible.

From what I’m seeing, the regular color (at least in the Synapse color palette created a while back) is #515359 and the dark color is #1A1C29. Given this, I’d propose #1A1C29 to be used unless another color is currently being used that’s still AAA compliant, and we can readdress this in the next week or so when I have an updated typography palette for Synapse.

Let me know what you think and if you have any questions or concerns.

Done
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Nick Grosenbacher

Reporter

Duminda Aluthgamage

Validator

Duminda Aluthgamage

Priority

Major

Labels

Sprint