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

bug/#616 - small layout fix for search panel #634

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

monsieurtanuki
Copy link
Contributor

Impacted files:

  • search_panel.dart: added margin to title
  • smooth_app/pubspec.yaml: unrelated library upgrades
  • smooth_ui_library/pubspec.yaml: unrelated library upgrades

This is what it looks like in French:
Screenshot_2021-10-26-10-16-14

This is what it looks like in debug phase:
Screenshot_2021-10-26-10-15-19

Impacted files:
* `search_panel.dart`: added margin to title
* `smooth_app/pubspec.yaml`: unrelated library upgrades
* `smooth_ui_library/pubspec.yaml`: unrelated library upgrades
@monsieurtanuki monsieurtanuki linked an issue Oct 26, 2021 that may be closed by this pull request
1 task
@monsieurtanuki
Copy link
Contributor Author

Thank you @slava-sh!

For the record smaller titles are centered:
Screenshot_2021-10-26-12-20-12

And perhaps it would look better with the title aligned to the field hint instead of aligned to the RRect:
Screenshot_2021-10-26-12-24-46

If someone feels like fine-tuning the alignment, the margin in SearchPanelState would be

margin: const EdgeInsets.only(
  left: LARGE_SPACE + 20,
  right: LARGE_SPACE + 20,
  bottom: SMALL_SPACE,
),

@monsieurtanuki monsieurtanuki merged commit 7f665d1 into openfoodfacts:develop Oct 26, 2021
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.

Layout issue in the scan card due to the French translation
2 participants