Page MenuHomeWildfire Games

Adjust "Watch Replay" button
ClosedPublic

Authored by Langbart on Jun 13 2021, 6:14 PM.

Details

Summary

To avoid the line break for the Watch Replay button it needs to be quite long especially for the Greek language. I think the best solution might be to keep the line break and just increase the height for the Watch Replay and Continue button.

Test Plan

You could apply the patch and check if it looks good for all languages.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Langbart created this revision.Jun 13 2021, 6:14 PM
Owners added a subscriber: Restricted Owners Package.Jun 13 2021, 6:14 PM
Langbart requested review of this revision.Jun 13 2021, 6:14 PM
Langbart updated this revision to Diff 18256.Jun 13 2021, 6:38 PM

To avoid the line break for Greek you would have to make the buttons even longer, it might be a better solution to focus more on height and keep the line break.

Langbart edited the summary of this revision. (Show Details)Jun 13 2021, 6:44 PM
Langbart edited the test plan for this revision. (Show Details)

Tried it for different languages (not all) and found no problems. Text fits nicely in the box / or at least better than before, in the case of greek.

marder accepted this revision.Jun 13 2021, 8:58 PM
This revision is now accepted and ready to land.Jun 13 2021, 8:58 PM
bb accepted this revision.Aug 29 2021, 10:29 PM
bb added a subscriber: bb.

Tried it for different languages (not all) and found no problems. Text fits nicely in the box / or at least better than before, in the case of greek.

There is the longStrings language too which you can generate: running python3 source/tools/i18n/generateDebugTranslation.py --long will generate a new language containing for every string the longest translation among all languages, this way you can test a whether gui-items are large enough for all languages.

If one searches a bit with that one will find many more places with a similar issue. Don't want to fix the world here, (so accepting), but feel free to make patches for it (grouping some together is fine, but keep the patches reasonably small for review).

longstring is happy about this button now.

binaries/data/mods/public/gui/summary/summary.xml
186 ↗(On Diff #18256)

height of 37 pixels seems weird, but can't improve upon it.

This revision was automatically updated to reflect the committed changes.