Hide Forgot
Description of problem: Apper currently ships with a downstream hack (apper-0.9.2-style.patch) that makes it refuse to use the systemwide Qt widget style. This was done to workaround a harmless, purely cosmetic glitch (bug #1209017). But it causes a much worse cosmetic glitch: the application looks just completely off. Just compare the big progress bar on a stock Fedora 24 installation (where Oxygen is not available, and thus Cleanlooks is used) with a Breeze progress bar. It just looks completely wrong. Native KDE applications must not be disguising themselves as GTK+ applications when running under Plasma. Version-Release number of selected component (if applicable): apper-0.9.2-6.fc24 How reproducible: Always Steps to Reproduce: 1. Use Apper. Actual results: Uses an ugly look&feel. Expected results: Looks&feels like a native KDE application. Additional info: If you care so strongly about the bug #1209017 glitch, please fix it properly, or get the Breeze developers to fix it properly. (It is probably a bug in their style.) It is absolutely not acceptable for every single application to work around style bugs by hardcoding some style. That will only lead to an inconsistent hodgepodge of differently-looking applications, and by hiding the real problem, it also makes it unlikely it will ever get fixed.
I also disagree with the comment on your patch claiming it is "upstreamable". Such gross hacks are absolutely NOT acceptable in upstream code (nor downstream, IMHO).
Oh, and the bug #1209017 glitch has been there for years, probably for as long as Apper exists at all, with the Quarticurve theme. I have never seen an Apper without that glitch. It is perfectly usable anyway.
Sorry if I marked the patch upstreamable, I agree that it is not. That said, I thought we were doing to discuss this first onlist and only move to change things if there's a consensus? either way, my personal opinion is still the same: that forcing the use of a known good/working style is preferable to layout issues (that sometimes make text unreadable).
At the very least, you should add something like Krita's KRITA_NO_STYLE_OVERRIDE (see line 162 here: https://quickgit.kde.org/?p=krita.git&a=blob&h=4d43185fc5d7a388ddde274bc04ffc48b2cdfccd&hb=044694c89b3511e0b4f00ded11c2be12d42eaee8&f=libs%2Fui%2FKisApplication.cpp ). But I think Apper would really be better to use the correct style by default. People who want to override it can do it (just use the menu editor and add --style=whatever). This doesn't need to be hardcoded in the code at all.
(In reply to Kevin Kofler from comment #4) > But I think Apper would really be better to use the correct style by > default. People who want to override it can do it (just use the menu editor > and add --style=whatever). This doesn't need to be hardcoded in the code at > all. Most users will have no idea that they can override the style to make the text legible, or that the garbled text isn't actually very useful, they just see illegible text and don't know what it says. The default for regular users should be legible text. If the inconsistent style hurts your eyes then use the environment variable, don't force your preference for style over substance on everyone else.
> That said, I thought we were doing to discuss this first onlist and only move > to change things if there's a consensus? I thought you wanted to do it on IRC, and since this was not discussed in an IRC meeting (this week's meeting was skipped instead), I figured I need to file this bug to get things moving. But here's it on the mailing list: http://www.spinics.net/linux/fedora/fedora-kde/msg18097.html
> Most users will have no idea that they can override the style --style=… is a standard parameter for all Qt apps. It is a lot more discoverable than an undocumented application-specific environment variable, or worse, having to recompile the code (or patch the binary, ewww!) as it stands now. > they just see illegible text and don't know what it says. Now instead they see an application that looks bad, which makes them think, consciously or subconsciously, that Fedora is a bad distribution because it cannot even make KDE applications look consistent with each other. Even if they do not immediately notice the wrong theme, it still subconsciously looks "messy". > The default for regular users should be legible text. 99% of the text in the app is legible. It is only that progress status (that is not even correct with PackageKit-hif to begin with!) that is not. > If the inconsistent style hurts your eyes then use the environment variable, > don't force your preference for style over substance on everyone else. It is not a preference for style over substance, it is a preference for a local cosmetic glitch (a detail looks broken) over a global one (the whole application looks broken).
bz is not a good forum for discussion and debate, please followup onlist per comment #6
Grrr, still no update here, after a month! I ran: sudo sed -i -e s/KDE_FULL_SESSION/KDE_FOOL_SESSION/g /usr/bin/apper to stop it from looking like crap here. And ironically, the Cleanlooks style your hack uses instead actually has much worse graphical glitches. In the Cleanlooks style, if I click on a package, the description does not render until I click inside the text, or where the text should be (a glitch I also had with Quarticurve – Quarticurve also had the same glitch with the overlapping progress text as Breeze though). With Breeze, showing the description just works. We really need to kill that style override.
For what it's worth, I cannot reproduce the problem you describe in comment #9 about updates text not rendering (until you click it). (though honestly, PK lot longer to load the data than expected... )
You probably have the Oxygen Qt 4 style installed. I don't (this is a fresh F24 install on a new computer), so your hack uses Cleanlooks instead.
I know, I did purposely remove oxygen first (and indeed cleanlooks very different than oxygen)
OK, I can reproduce this now, and I agree that's pretty horrible. :( In the end, the only style that works well still seems to be oxygen. So, I think I'll go with: 1. modify patch to only try using alternative style oxygen (if available) 2. add Recommends: qt?-style-oxygen
apper-0.9.2-7.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-f5295dc586
Your fix looks acceptable (with also the opt-out environment variable), I'd still like the hack to just go away though. :-)
apper-0.9.2-7.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-f5295dc586
apper-0.9.2-7.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.
apper-0.9.3-0.4.20161109.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-42e3aac51f
apper-0.9.3-0.4.20161109.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-42e3aac51f
apper-0.9.3-0.4.20161109.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.