| Summary: | Lancelot's main panel icon doesn't enlarge when the panel height increases | ||
|---|---|---|---|
| Product: | [Unmaintained] plasma4 | Reporter: | Xavion <Xavion.0> |
| Component: | widget-lancelot | Assignee: | Ivan Čukić <ivan.cukic> |
| Status: | RESOLVED NOT A BUG | ||
| Severity: | normal | CC: | Xavion.0 |
| Priority: | NOR | ||
| Version: | 4.9.3 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
Smaller panel height, larger Lancelot icon
Larger panel height, smaller Lancelot icon |
||
|
Description
Xavion
2012-11-19 23:18:14 UTC
Created attachment 75365 [details]
Smaller panel height, larger Lancelot icon
Created attachment 75366 [details]
Larger panel height, smaller Lancelot icon
The height of the panel in the second screenshot is larger, yet the size of the main Lancelot icon is actually smaller. You should probably just copy the relevant panel icon expansion code across from Kickoff for consistency. *** This bug has been marked as a duplicate of bug 309767 *** Uh, wait, I read wrong. Reopening. You can set the icon size at: System Icons -> Application Appearance -> Icons -> Advanced -> Panel Unlike the bug 309767, lancelot's behaviour has not changed. I can confirm that setting the panel icon size manually does fix the problem. However, why don't you want Lancelot's main icon to resize dynamically like Kickoff's does? For me, it behaves exactly the same. But, even if it doesn't - rationale behind it is two-fold: - if we have this setting, we should obey it (most of the icons did so until bug 309767 - people that use vertical panels tend to make them wider to be able to show the window titles. Imageine a 200x200px lancelot icon in there I hear what you're saying, but I still think Lancelot should behave like Kickoff in this way. Most people won't realise that they need to set the panel icon size manually. This is mainly because that setting is hidden away somewhere in the System Settings. Perhaps there should be a link to it in the More Settings section of the Panel Toolbox. It *is* a bit hidden config option, that I agree. I don't, however, think that whenever other applets change the previously established behaviour, that Lancelot should follow. (especially when there are users that don't support the change in the first place). For me, the current behaviour (that evolved from scale-and-take-as-much-space-as-possible through a few iterations here on bugs.kde.org) is the sane one. I'd need a really big reason for why it is bad in order to change it. 'It doesn't behave like kickoff' is not a reason I'd usually even consider - Lancelot was made not to be kickoff. |