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

Improve namespace-filter expansion UX. #3555

Closed
foot opened this issue Jan 10, 2019 · 3 comments · Fixed by #3562
Closed

Improve namespace-filter expansion UX. #3555

foot opened this issue Jan 10, 2019 · 3 comments · Fixed by #3562
Labels
component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer estimate/hours It will take less than 8 hours to implement feature Indicates that issue is related to new end user functionality

Comments

@foot
Copy link
Contributor

foot commented Jan 10, 2019

#3117 made having a lot of namespaces more manageable in the UI.

There were some ideas to improve this even further:

  • @AVRahul:

    there are some improvements which could be better, the hovering effect can be just restricted to the last-child class instead applying in overall sidebar.

  • Provide an explicit "show more" button to expand and collapse the namespace list.

@foot foot added component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer feature Indicates that issue is related to new end user functionality estimate/hours It will take less than 8 hours to implement labels Jan 10, 2019
@dholbach
Copy link
Contributor

@AVRahul: did you have a different solution than #3117 in MayaOnline? Or improvements to it?

@AVRahul
Copy link
Contributor

AVRahul commented Jan 17, 2019

sorry for the delay, @dholbach as per the current scenario in MayaOnline we have improved as mentioned above. We have restricted the hover effect just to last-child class to maintain better experience in all the browsers. Because when we checked it in Firefox, the scrollbar appears even when there are fewer namespaces as shown below. So we fixed this issue.

screenshot

@dholbach
Copy link
Contributor

@AVRahul do you think you can put the PR together, so Simon and Filip can take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer estimate/hours It will take less than 8 hours to implement feature Indicates that issue is related to new end user functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants