-
Notifications
You must be signed in to change notification settings - Fork 9
-
Notifications
You must be signed in to change notification settings - Fork 9
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
multiline warnings containing links are difficult to click due to resizing #51
Comments
First, hover seems incompatible with how the directive generates the page. This necessitated the click to increase size fix. I also find hover changes to not work well on mobile devices. I believe this issue is rooted in your comment here: asdf-format/asdf#1337 (comment). First note that sphinx structures things so that Indeed, this is exactly how sphinx-asdf/sphinx_asdf/static/custom.css Lines 11 to 13 in da575fb
warning .
As for the reason why it has to affect all |
Note these changes were first developed as part of asdf-format/asdf#1142 |
After looking at this a bit more my vote is for removal of custom.css unless there are other uses aside from the resizing. The resizing also interferes with selecting content within blocks. As one example, for the asdf_library: !core/software-1.0.0 {author: The ASDF Developers, homepage: 'http://github.com/asdf-format/asdf', Removing custom.css allows for normal use of the scroll bars in the blocks. The resizing also does not consistently work on mobile and touches sometimes result in scrolling and other times result in resizing that resets the scrolling. |
Then remove it. |
The active resizing to allow for easier reading of asdf file contents limits interaction with links in warning block text.
sphinx-asdf/sphinx_asdf/static/custom.css
Line 7 in da575fb
Perhaps a hover would allow for both uses but would require testing.
The text was updated successfully, but these errors were encountered: