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

Fix: missing tooltip translation & filter editor wrapping #1305

Merged
merged 2 commits into from Aug 1, 2022
Merged

Conversation

shamoon
Copy link
Member

@shamoon shamoon commented Jul 29, 2022

Proposed change

This small PR addresses two (unrelated) issues reported in 1.8.0, the first a missing translation string and the second the fact that the filter editor text field can become too short on certain screen sizes. Regarding the second issue, fix was to increase the breakpoint at which everything goes to one line from 1200 --> 1400 px (e.g. at user's screen size things will now be on two lines). Screenshot below is new minimum width to illustrate, its still small but acceptable I think.

Screen Shot 2022-07-29 at 2 42 02 PM

Fixes #1299
Fixes #1300

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please explain)

Checklist:

  • I have read & agree with the contributing guidelines.
  • If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
  • If applicable, I have checked that all tests pass, see documentation.
  • I have run all pre-commit hooks, see documentation.
  • I have made corresponding changes to the documentation as needed.
  • I have checked my modifications for any breaking changes.

@shamoon shamoon added bug Bug report or a Bug-fix frontend labels Jul 29, 2022
@shamoon shamoon added this to the Next Release milestone Jul 29, 2022
@shamoon shamoon self-assigned this Jul 29, 2022
@shamoon shamoon requested a review from a team as a code owner July 29, 2022 21:50
@shamoon shamoon changed the title Fix/1299+1300 Fix: missing tooltip translation & filter editor wrapping Jul 29, 2022
This was linked to issues Jul 29, 2022
@tooomm
Copy link
Contributor

tooomm commented Jul 30, 2022

Looking at your screenshot, it's still small indeed. Probably even smaller for other languages...
Could there be a minimum size for the search area, where it no more shrinks but pushes the other buttons to the second line?

On mobile, it switches to icon-only filter buttons at one point. Maybe that's a reasonable solution for smaller desktop/laptop/tablet screens as well?

@qcasey qcasey modified the milestones: Next Release, v1.8.1 Jul 31, 2022
@qcasey
Copy link
Member

qcasey commented Aug 1, 2022

Looking at your screenshot, it's still small indeed. Probably even smaller for other languages...

This is true, unfortunately this PR doesn't fix for other languages (try German). The longer titles of the other items push the field small once again.

localhost_8000_settings

@shamoon
Copy link
Member Author

shamoon commented Aug 1, 2022

Hmm, actually should be OK in German, that’s where I tested it. But I will double-check.

We discussed this a bit in the original Storage paths PR, it was always going to be a trade-off. The “fix” (barring a more sig UI change is just pushing the limit at which we keep things on two lines (eg currently 1200, PR goes to 1400, could go further…) but yea, it’s a compromise.

But I will double-check this a bit later

@qcasey
Copy link
Member

qcasey commented Aug 1, 2022

But I will double-check this a bit later

OK thanks, let us know might be just my chromium.

pushing the limit at which we keep things on two lines (eg currently 1200, PR goes to 1400, could go further…)

As @tooomm suggested would a min-width fix longer languages while not sacrificing shorter ones? See these two screenshots with 150px min width on the input, in German the Reset Filters button gets kicked to a second line while it's unchanged in English.

Screenshot from 2022-07-31 20-11-29

Screenshot from 2022-07-31 20-12-02

@shamoon
Copy link
Member Author

shamoon commented Aug 1, 2022

Yea I think this is the best compromise, thanks. Here's a screenshot around 1450px on German and English with newest changes:

Screen Shot 2022-07-31 at 8 23 24 PM

Screen Shot 2022-07-31 at 8 25 54 PM

Copy link
Member

@qcasey qcasey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks for the fix

@qcasey qcasey merged commit da99ba1 into dev Aug 1, 2022
@qcasey qcasey deleted the fix/1299+1300 branch August 1, 2022 03:40
This was referenced Aug 1, 2022
@shamoon shamoon modified the milestones: Future Release, v1.8.1 Aug 2, 2022
@github-actions
Copy link
Contributor

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Bug report or a Bug-fix frontend small-change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Missing translation [BUG] Input field too small
3 participants