FooSoft / yomichan

Japanese pop-up dictionary extension for Chrome and Firefox.
https://foosoft.net/projects/yomichan
Other
1.07k stars 229 forks source link

Settings v2 feedback #1000

Closed toasted-nutbread closed 3 years ago

toasted-nutbread commented 3 years ago

This issue is for any questions/comments/feedback related to the new settings page.

Changes:

toasted-nutbread commented 3 years ago

Quoting feedback from @FooSoft https://github.com/FooSoft/yomichan/issues/1079#issuecomment-739554939:

  • The "Show popup preview" toggle jumps to top of the screen when enabled and returns to screen center when disabled. Not sure if it's supposed to do this or if it is a bug (I am getting this on Chromium). When expanded, there is also a lot of vertical white space inserted between the elements. We should probably have the preview contents be right below the toggle and not have the toggle jump around depending on its state. Also, would there ever be a time where the user would not want to display the preview at all?

The reason it jumps is because the <iframe> of the preview isn't visible until it's enabled, causing a layout change. I tested an animation on the frame to prevent the immediate jump (but still moved to top), but there were other issues related to jitter. You're probably right that it may be easiest to have it always visible.

  • The link for "Top" which appears after you scroll downwards appears in an area which is already pretty busy and this make it hard to notice. I'm wondering if it would look better in bottom right corner of the screen, as there is nothing currently there and it would be easy to spot. I get that having "top" be at the top of the page makes sense, but in its current form I think a lot of people will not notice it.

May also be viable to just remove it altogether, since it's kind of redundant with the scrollbar or keyboard navigation.

  • The popups (Profiles, Dictionaries, etc.) have headings which do not stand out enough IMO. I'm not sure if there's a better style to use here, but there should be something to make them stand out a little better.

They do kind of blend in; I wonder to what extent that might affect usability. The design is similar to the design of Chrome's settings page, and it uses this same design. A few things I can think of:

The popups always open in full-height which can seem to be a bit overkill if there is not a lot of info to display.

The full-height ones were sized that way based on a use case where there is a lot of info in them. The two main ones which may contain "dead space" may be:

I could have also gone the route of making the height expand based on content, but I chose not to do that for now because of how it affects button/input positions as entries are added/removed.

  • We can probably get rid of the whole About/Donate/whatever thing. There's really not really a lot that needs to be said "about" Yomichan and considering that this is a more distributed development effort at this point, having a "donation" thing there is strange too, especially considering that project has no direct monetary expenses at this point.

  • ...with that being said, maybe have some sort of "Help" or "Report a bug" or something link that sends the user to GitHub?

I plan on having a general info page which shows links to the homepage, Github, version info, licenses, extension permissions, etc. So it won't be completely removed, but it will be changed to something with more info. The ? button on the action popup will probably also direct to that.

Thermospore commented 3 years ago

I don't have anything particularly insightful to say, but I absolutely love the new settings page! The formatting and organization is great, and I'm glad to see more info text was added as well. Nice work, guys!

Thermospore commented 3 years ago

I just noticed the donations button/section is missing

also this button doesn't seem to do anything (maybe it's just unimplemented) image

toasted-nutbread commented 3 years ago

Correct, it's unimplemented in the released version and implemented for the next release.

PhantomAvery commented 3 years ago

This update is really great and the new settings are very easy to navigate. I especially loved playing around with the 'Use a native browser window instead of an embedded popup' and 'Open a native browser window when copying Japanese text' options, which I hadn't noticed before. There are just a few things that I would love to see added to the advanced options pages: 1: The option to close pop-up windows automatically once the scanning key is no longer being pressed.

2: The ability to hide the pop-up window by pressing the scanning key on areas of websites where there is no text.

3: The option to change the text size/scale of the dictionary content when using Yomichan in a native browser window.

4: A toggle for the blue-colored text of selected dictionary entries, or the ability to change the color.

Other than that, this update has been really great! Thanks for all the good work and happy holidays!

Thermospore commented 3 years ago

I second #⁠2! Forgot to mention it

toasted-nutbread commented 3 years ago

2: The ability to hide the pop-up window by pressing the scanning key on areas of websites where there is no text.

  • This feature, which I believe was present in past versions, was a huge draw for me. I loved the convenience of being able to hide the pop-up window without having to click, as it allowed me to watch subtitled content and use Yomichan without playing or pausing the video. I really loved this feature and I am finding it quite inconvenient using Yomichan with this feature not working.

Does this not work when the Automatically hide results/Auto-hide search popup option is enabled? That works for me, although you do have to move the cursor while holding the keys, which should have always been the case. Or am I misunderstanding the ask/what's going wrong?

Note: the description for this option on the settings v2 page is wrong; it should apply even when a modifier key is assigned. (#1150)

Thermospore commented 3 years ago

Enabling that setting returns behaviour to how it was before, thanks!

I assume the behaviour of the option was changed (and the text just wasn't updated to reflect it)? I don't recall changing it

toasted-nutbread commented 3 years ago

1: The option to close pop-up windows automatically once the scanning key is no longer being pressed.

  • I love having this 'auto-hide' feature for instances where there is no key required to scan text. I would love to see this option added as well for situations where there is a scanning key so that the pop-up may automatically hidden after a user-inputted amount of time!

This is already possible, although you have to set it up using the advanced scanning options. Here is an example:

image

Inputs 1 and 2 are the defaults, adding a 3rd and configuring it as shown in the image will make the popup hide when Shift is not held. (Use the kebab menu to show the advanced options)

toasted-nutbread commented 3 years ago

3: The option to change the text size/scale of the dictionary content when using Yomichan in a native browser window.

  • I like using Yomichan in dark mode, but this can make it difficult to read complex Kanji ...

Is this just due to the size, or something else? Is this specific to dark mode, or anything that you think could be improved?

  • ... so I am very happy that on this new settings page, more options have been given to change the popup scale. ...

Technically the options were there on the v1 settings page, but not as well designed/organized.

  • ... I would love to see customizable options like these added for the native browser window version of Yomichan as well, as I find the text a bit difficult to read.

I can probably set up this option (or a different one) to control that as well. Native browser zoom functionality also works (CTRL++, CTRL+-, CTRL+mousewheel), although that does affect other Yomichan extension pages.

4: A toggle for the blue-colored text of selected dictionary entries, or the ability to change the color.

  • The blue color currently used for selected dictionary entries is quite hard to read on dark mode. I don't mind the idea, but I would love to see more options for users to control the functioning of this feature.

This was a bug and is fixed for the next patch release. The color can be adjusted by using the custom CSS option, although you would have to have some knowledge of how to edit CSS.

PhantomAvery commented 3 years ago

Is this just due to the size, or something else? Is this specific to dark mode, or anything that you think could be improved?

Mostly the issue is just the size. Although I do think dark modes tend to make text more difficult to read in general, it's not something I'd fault against the program itself. Now that I'm trying the CTRL++ functions, I'm finding it works really great! Thanks very much for the help!

FirionHope commented 3 years ago

Bit late but wanna say that the new settings menu looks amazing, much easier to understand

fmy1993 commented 3 years ago

image

hi,the tool is good ,but the import button is grey, I do not konw why

toasted-nutbread commented 3 years ago

hi,the tool is good ,but the import button is grey, I do not konw why

Likely your Anki card settings are not configured. Use the Configure Anki card format… button in the Anki section to set this up.

Aridow commented 3 years ago

Is it possible to visually reorder the dictionaries, instead of just changing the priority numbers?

toasted-nutbread commented 3 years ago

Yes, I'm working towards supporting that; #1587. Will probably come with #1539.