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
Add-on store: Title and searchbar focus tweaks #2406
Conversation
Job #1711: Bundle Size — 11.01MiB (~-0.01%).Warning Bundle contains 19 duplicate packages – View duplicate packages Bundle metrics
Bundle size by type
View job #1711 report View jimtng:addons-store branch activity View project dashboard |
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
I don't think this is a good idea, as having tooltips on clickable links is usually not working very well with touch devices. |
This reverts commit e841264. Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Reverted. |
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
BTW, the little buttons on the bottom of the blockly editor screen also have tooltips. |
Yes, but these buttons are also displayed on desktop, where the tooltip is usable. The toolbar of the store is only displayed on mobile, where it is not really usable. |
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.
Thanks for the PR, however I think we can achieve the same without adding more data to addon-store.js. I will push a commit for that.
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
- Add tooltip to the bottom tabbar on narrow screens