-
Notifications
You must be signed in to change notification settings - Fork 112
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
Popup UI improvements #508
base: master
Are you sure you want to change the base?
Conversation
Current status: 2024-01-06.12-09-38.mp4 |
View Playwright Report (note: open the "playwright-report" artifact) |
Very cool stuff! However I feel like the wide popup is somewhat obtrusive and probably wouldn't be favored for general use. Would this be a separate view that could be entered into? I've been thinking about how better to display dictionaries in a small popup and have been thinking about potentially using tabs or a dropdown menu or even a custom scroll bar of some sort, as I find the main pain point is scrolling between definitions and use alt + scroll to navigate. |
potential enhancement: #77 |
the only thing i actually like about this is the increased separation of dictionaries with the greater use of horizontal space, wider name headers, and more color. the horizontal separation won't work on a phone, which is fine, but i think column count shouldn't be purely inferred from width. limit variables on column count and width would make it configurable. i'm concerned about how the color tinting would look in dark mode. when i'm reading at night i need the background color to be pure black and the foreground colors to have limited contrast because my screen doesn't dim enough. the missing scrollbar, round borders, transparent gaps, and overflow fade are viscerally disgusting to me. seeing the background site leaking through the gaps between entries as you scroll is incredibly distracting. i don't think any modern browser or os ui uses transparency to separate elements, this is just rice. i also have the action bar always on on my phone so i can always use the close button in order to select japanese text, and i don't see how the action bar would work at all without the frame being a solid window. how do collapsible dictionaries look? some other concerns i'm noticing from the video: images being displayed inline causes the vertical size of the whole row to explode. a term containing less dictionaries than the column count results in wider columns, which looks especially weird when the colored header extends across the entire width, but the dictionary name and entry are both very small and to the left. |
@praschke I appreciate the feedback, but the way its written sounds a bit harsh. 😅 |
@djahandarie i'm sorry for shooting off the cuff. i genuinely love the improved use of horizontal space, but the translucency effects seem like a primarily aesthetic change. i also don't see how the popup resizer or the action bar will work with this when the popup itself is missing. if the translucency effects can be configured away i promise i won't care. rounded borders and missing scrollbars shouldn't have been included in that list because they are consistent with modern browser and os ui, and can probably be changed with custom css anyway. |
Rounded borders are modern right now in 2024, but I swear there is a perpetual cycle with UI design where the general consensus switches from "rounded good" to "square good" every several years, so take that with a grain of salt. With regards to scrollbar, keep in mind situations where someone may not have a touchscreen or a mouse wheel, which are rightfully rare these days, but still. |
Looks like an amazing view for the large screen where it's possible to utilize all this space! Not quite sure about removing the scrollbars though. The colors and corners look much more modern and cool than the current yomitan appearance. |
I'm starting to do a popup UI overhaul for Yomitan, with some of the following goals:
High-level concerns:
Remaining items: