Details
Details
The 350px size works for these elements since they are all in the first 400px of the page (thus are compatible with 1024x768).
Notice we can't add it to the ModernDropDown style, because it wouldn't suit many dropdowns in the lower half of the screen.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 1829 Build 2901: Vulcan Build Jenkins Build 2900: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1273/ for more details.
Comment Actions
It looks good to me. I would've liked the player menu to be wider too, but I understand in the 1024 x 768 mode there's no horizontal space left for that. So this is fine, thanks.
Comment Actions
That should still be done, but in a separate patch, needs more thinking (and most likely a screensize dependent solution, relative width, additional testing)
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1274/ for more details.