-
Notifications
You must be signed in to change notification settings - Fork 14
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
The list content to move as it's being scrolled down. #3
Comments
Sorry for the force pushes! Edit: ah, ya mean with the tools anchored to the right, like a dock? |
Alright, so in 411f08a I just made the In d4ec2e5 I tried making the controls I currently favor the first approach because it was much simpler, but the issue with each is that they both break the |
Both 411f08a and d4ec2e5 are actually great candidates for this issue i think, at least in their current forms they're usable. I think if 411f08a had a screenshot mode of some sorts where you could see the whole list at once, it is a perfect solution, making it easier to navigate and make the actual list. |
I think making the tier list a scroll box is my preferred solution, but I'm open to whatever works best. Thanks for the help ✌🏻 |
So ^that's how I'd impliment a toggle for it, just needs an image for the button, and the box of images should still be updated to scroll. This makes the parent viewport static (ie. no element expands the page when overflowing, unless someone puts in a really long title?). Still considering the |
lol
Non-issue, works like that already |
MIT Licensed, kinda verbose on both the HTML and CSS fronts...
|
Hello there, it is currently an issue that when you add more than a few rows to the list, it becomes hard to add stuff as you have to go all the way to the top in order to select your image and move it down. It would be pretty good if the left images box had a size limit, and could be scrolled up and down in order to view other images. It would also be pretty good if the box would move as it's being scrolled up and down.
The text was updated successfully, but these errors were encountered: