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
General browser fixes for collapsed sidebar #914
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Maybe you could even add a test to check the arrow direction XD
'-scale-100': isCollapsed, | ||
})} | ||
/> | ||
{isCollapsed ? ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dottorblaster So I am now using two implementations of the EOS Keyboard Double Arrow
icons but it @arbulu89 had a suggestion to create a React component for both icons so that we can reduce the repeated attributes for size, color and classname. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jagabomb a component would be an overkill to me, maybe we can try just defining these things as const
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dottorblaster Cool, I have defined the three const
values in a new commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go!
* General browser fixes for collapsed sidebar * Formatted using Prettier * Added EOS Keyboard Double Arrow Right icon * Changed position of title tag for Sidebar menu items * Added three const values for sidebar icon color, size and classname
Description
Minor improvements to remove issues in Safari browser with collapsed sidebar elements. Also changed implementation of Double Arrow Left icon to library standard due to fix with icon library.
Fixes:
Issue with icon not changing direction when the sidebar is collapsed in Safari.
Alignment of Trento logo not being centred when the sidebar is collapsed in Safari.
How was this tested?
Visually in Safari and Chrome