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

Scrollbar for WEG #351

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

luriusTM
Copy link
Contributor

@luriusTM luriusTM commented Dec 11, 2024

For this I got to reduce the resolution and add some items to dock, but these are the test videos:

Screen.Recording.2024-12-11.221159.mp4
Screen.Recording.2024-12-11.221250.mp4

--config-margin, --config-padding, --config-item-size, --config-item-zoom-size change are also work (while zoom size bigger than item size, if not it collapses...)

In overall:

  • 2 new div for the scroll
  • Weg items are dragable only in that axis which is currently presented.
  • Scroll is present when needed and containers are pixel precise.
  • Scroll X axis also on Y wheel.
  • multiple little style issue fixed because pixel precise calculations can be only done on perfect alignment...
  • Hoover animation is really blocked for dragged items.
  • Weg item touchbox is reduced with var(--config-margin) (it would go out from the box, this is a must have modification)

Bold items are must have, if any of that is about question, the whole concept cancelled and got to look for a different solution. (not built in scroll for this)

Although this has only some line modification, this was burnt days from my life...
image
The above small concept issue was so tiny I wouldn't recognised it 'til a day!

@luriusTM luriusTM force-pushed the feature/weg-scroll branch 2 times, most recently from e81d78f to f2e7bb4 Compare December 22, 2024 08:08
@luriusTM luriusTM force-pushed the feature/weg-scroll branch from f2e7bb4 to eead396 Compare January 1, 2025 21:18
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