Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redesign of the Sidebar of CARE #3781

Closed
joelsajichacko opened this issue Oct 18, 2022 · 5 comments · Fixed by #3843
Closed

Redesign of the Sidebar of CARE #3781

joelsajichacko opened this issue Oct 18, 2022 · 5 comments · Fixed by #3843
Assignees
Labels

Comments

@joelsajichacko
Copy link

Redesigning the sidebar for a better layout with proper spacing between the elements;

Font used; Inter

Link to Figma; https://www.figma.com/file/fqDMgeN2m4TCSz5KobZaKW/CoronaSafe-Network-Care?node-id=1584%3A719

image

@rithviknishad
Copy link
Member

@joelsajichacko @gigincg @aparnacoronasafe

How about having a similar background in the recently redesigned warranty card, be also used in the sidebar's background and the login page background?

Warranty Card's Background for reference:

image

to be used in:

image

cc: @skks1212

@joelsajichacko
Copy link
Author

@rithviknishad it's a good idea that the background pattern can be implemented on the login page.

For the sidebar, don't think its a good idea, because normally the sidebars are really made clean without any background patterns

@shivankacker
Copy link
Member

shivankacker commented Oct 27, 2022

@rithviknishad make sure you use i tags for importing unicons. It has already been added to care. Syntax is similar to font awesome
somethink like <i className="uil uil-[icon]"/>
all Icons can be found on the unicons website

@gigincg
Copy link
Member

gigincg commented Oct 28, 2022

@rithviknishad @skks1212 w.r.t. the Slack Discussion I think we decided not to use uni-icons? Or did we?

@rithviknishad
Copy link
Member

rithviknishad commented Oct 28, 2022

@gigincg Not to use. I'll be porting pf-icons implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants