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

📱 [#1140] mobile: Samenwerken-listview #481

Merged
merged 2 commits into from
Feb 21, 2023

Conversation

jiromaykin
Copy link
Contributor

@jiromaykin jiromaykin commented Feb 20, 2023

Responsive design: fix issue where header text and button should be on top of each other, not next to each other;
see https://taiga.maykinmedia.nl/project/open-inwoner/task/1140
⚠ Note: the current structure of templates, (where the entire H1 heading is surrounding the button), makes word-breaking behave strange again sometimes (causes letter-breaking in very long words on very tiny screens).

@codecov-commenter
Copy link

Codecov Report

Merging #481 (f7f505f) into develop (ccc62c3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #481   +/-   ##
========================================
  Coverage    96.46%   96.46%           
========================================
  Files          506      506           
  Lines        18300    18300           
========================================
  Hits         17654    17654           
  Misses         646      646           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jiromaykin jiromaykin marked this pull request as ready for review February 20, 2023 16:20
@jiromaykin
Copy link
Contributor Author

Apart from the word-break-row/column fix, I've fixed a typo in translations (without generating 'compilemessages' etcetera).

Copy link
Contributor

@vaszig vaszig left a comment

Choose a reason for hiding this comment

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

It looks better!
I am not sure about the change in translations. Alex can have a look as well.

Copy link
Contributor

@Bartvaderkin Bartvaderkin left a comment

Choose a reason for hiding this comment

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

Also not sure about the change in the .po: we usually don't generate or commit those

@alextreme
Copy link
Member

I don't mind too much about also updating the translations, I tend to do this at the end of each sprint in a separate branch/PR. Having multiple branches with each their own translation changes can lead to a mess, so I propose having a "Translation Ducky" at the office to avoid this.

@alextreme alextreme merged commit 0ca5a95 into develop Feb 21, 2023
@alextreme alextreme deleted the feature/1140-mobile-Samenwerken-listview branch February 21, 2023 10:09
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.

5 participants