| Summary: | some graphical glitches in the UI | ||
|---|---|---|---|
| Product: | [Applications] konquest | Reporter: | Majin Sniper <majinsniper> |
| Component: | general | Assignee: | Pierre Ducroquet <pinaraf> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version: | 2.0 | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
Screenshot 1
Screenshot 2 |
||
|
Description
Majin Sniper
2008-02-09 16:27:45 UTC
Created attachment 23503 [details]
Screenshot 1
Bar without textbox, yellow background overlapping the grid horizontally.
Created attachment 23504 [details]
Screenshot 2
Bar with textbox, blue box inside grid, yellow box fully overlapping, ¨Player
1¨ not colored.
I've got a fix for the colored label in my local copy, but I'll try fix the other problems before commiting. And it's gonna be quite hard. Konquest is using reals for the coordinates actually, and the problems you can see are rounding problems... If only screens could understand what a half pixel is... :) Sorry, I can't fix the bug. I fixed the label color bug easily (commit 772891, backport to KDE4.0 coming soon), but the coordinates problem has no fix. It's quite easy to understand. Say you have 16x16 sectors on the map. Say your map is 333x333 pixels on screen. Ideally, the sector size would be 20.8125 pixels. But it's impossible, so it's rounded. But when it's rounded, there is a problem : if it's rounded to 21 pixels, you don't have enough space on screen (or sectors will have to overlay), and if it's rounded to 20 pixels, there will be some space between some sectors or under/above the map. In both case, it's worse than what we have so far :/ I could hide the bug by removing the colored background on planets and instead do some effects on the planet icon itself, but I don't think it's worth it... So I mark this bug as fixed, because everything fixable has been fixed, and the unfixable problems are just waiting for screens understanding what floating point pixels are... (By the way, thanks for the bug reports) I think bug reporting is the least I can do :-). What was the most important part for me is the problem with the resizing of the top bar, when the textbox is shown. The re-rendering of the background svg is really slow, when the game is running in a maximised window here and seems not to be fixed :-/. I get the problem with the colored backgrounds of the planets. Maybe simply redrawing the grid around the planet would make it look a bit nicer? Thanks for the very fast response :) Please, next time try to split your reports in different issues, this way nothing will be lost... I've got a small idea to fix the top bar issue : instead of having the ship count "blinking" (and creating a serious slowness sometimes), I'll enable/disable it as needed... I'm having a weird issue so far with this change, but speed is much better as there is no more resizing of the whole area. SVN commit 772948 by ducroquet: Additionnal fix for bug 157531. Remove the top bar resizing issue : now, instead of having a disappearing line edit (and a lot of resizes because of this), we use setEnabled(true/false) on the line edit. It means much improved performances, no more bad looking resizes/repaints... Only good things. BUG: 157531 M +0 -4 gamelogic.cc M +9 -9 gameview.cc WebSVN link: http://websvn.kde.org/?view=rev&revision=772948 |