#2095 closed defect (fixed)
GUI: Language selection doesn't fit using 1x overlay
| Reported by: | joostp | Owned by: | eriktorbjorn |
|---|---|---|---|
| Priority: | normal | Component: | GUI |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
Using latest CVS, when using --force-1x-overlay the dropdown list of languages is too big to fit on-screen, so the bottom ones get cut-off (and you can't select them).
Maybe this can be made into a scrollable dialog? The attached screenshot demonstrates the issue.
Ticket imported from: #1243280. Ticket imported from: bugs/2095.
Attachments (2)
Change History (10)
by , 19 years ago
| Attachment: | gui-language-bug.png added |
|---|
comment:1 by , 19 years ago
| Summary: | GUI:Language selection doesn't fit onscreen using 1x overlay → GUI: Language selection doesn't fit using 1x overlay |
|---|
comment:2 by , 19 years ago
| Owner: | set to |
|---|
comment:3 by , 19 years ago
I've submitted a simple patch (#1263153) to work around the problem. It's not perfect by any means, but maybe it's good enough for 0.8.0? Fingolfin should probably have a say in the matter, though, since he's done most of the work on the GUI as far as I know.
In the meantime, I'm attaching a screenshot of what it looks like with the patch. The string "Norwegian Bokml" isn't rendered correctly, but that shouldn't have anything to do with the patch. The small font probably lacks the letter "".
comment:4 by , 19 years ago
Well, if it works fine, feel free to commit it. I still prefer the scrolling menu solution, however, the major advantage with your solution is that it actually is already implemented and working... :-)
Thanks for your work!
comment:5 by , 19 years ago
I agree that scrolling would be nicer but I figured that would be harder to implement. It would also mean having the change the behaviour of the pop-up dialog: It is currently removed when the mouse button is released, which obviously would make it hard to scroll. I figured that the patch might be good enough for the next stable release, and a better way can perhaps be implemented later.
I'm out of town for the weekend though, and won't be able to commit it until I return. If someone else wants to do it before then, that's fine with me.
comment:6 by , 19 years ago
| Owner: | changed from to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:7 by , 19 years ago
I've applied the patch, but do feel free to implement scrolling any time you feel like it.
comment:8 by , 6 years ago
| Component: | → GUI |
|---|
Demonstrating GUI issue with 1x overlay