Skip to content

Conversation

@wesruv
Copy link
Contributor

@wesruv wesruv commented Aug 5, 2020

Refactored open and close dropdown code
Added Esc key functionality
Updated variable and function names to be more consistent and easier understand
Added height calculation JS to secondary link dropdowns, but isn't working yet
Fixed bug where height of dropdown wasn't correct on resize
Fixed animation for dropdowns
Made sure 'heavy' DOM manipulation/reading isn't done all the time
Added dev only console logging when heavy processes are run
Moved more HTML class/id setup to JS instead of making stakeholders do it
Made sure words in the top bar don't wrap
Fixed jumping text
Fixed bug where single column didn't have padding at smaller breakpoints

Refactored open and close dropdown code
Added Esc key functionality
Updated variable and function names to be more consistent and easier understand
Added height calculation JS to secondary link dropdowns, but isn't working yet
Fixed bug where height of dropdown wasn't correct on resize
Fixed animation for dropdowns
Made sure 'heavy' DOM manipulation/reading isn't done all the time
Added dev only console logging when heavy processes are run
Moved more HTML class/id setup to JS instead of making stakeholders do it
Made sure words in the top bar don't wrap
Fixed jumping text
Fixed bug where single column didn't have padding at smaller breakpoints
@wesruv wesruv merged commit de5c7be into patternfly:CPFED-3689-new-navigation-epic Aug 5, 2020
@wesruv wesruv deleted the CPFED-3790--CPFED-3791--CPFED-3792-lots-o-features-n-bug-fixes branch August 5, 2020 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant