#7177 closed enhancement (fixed)
Deskbar should disable expand options in preference window when not in expando mode
| Reported by: | jscipione | Owned by: | tqh |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Applications/Deskbar | Version: | R1/Development |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
"Show Application Expander" and "Expand New Applications" in the Deskbar preferences only work in Expando mode, however, this is not communicated to the user. This ticket contains a patch which disables these options in the preference window when not in expando mode and re-enables them in expando mode.
Attachments (1)
Change History (10)
comment:1 by , 13 years ago
| patch: | 0 → 1 |
|---|
by , 13 years ago
| Attachment: | Deskbar Disable Expand Options 1.diff added |
|---|
comment:2 by , 13 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → in-progress |
follow-up: 6 comment:4 by , 13 years ago
Sorry for commenting on a closed ticket. The patch was just too quickly applied... Now that's something you don't read too often... :)
I was wondering if a tooltip on the disabled option would be appropriate: "Expanding is not available with Deskbar in horizontal mode." Or "bar mode"? Something like that...
comment:5 by , 13 years ago
I don't agree with you:
- it improves the behaviour over the old way
- it is clean code
- it is easy to change the behaviour based on test and feedback.
- further changes can be reported in new tickets
Development should happen in the version control system, not outside it. How often do you download and test patches and give feedback here?
comment:6 by , 13 years ago
Replying to humdinger:
I was wondering if a tooltip on the disabled option would be appropriate: "Expanding is not available with Deskbar in horizontal mode." Or "bar mode"? Something like that...
Yes, something like that would indeed be nice.
To tqh: I think you managed to misread humdinger's comment; we was not criticizing you for applying the patch so fast, he just mentioned that as an excuse that he didn't manage to comment before it was applied.
comment:7 by , 13 years ago
I didn't take it as critique, although I can see how my reply might suggest that.
I don't think I misread him, I was just explaining that it should go into the development tree if it is ok and further changes like the ones Humdinger want can be taken from there if it has already been landed.
Discussion and feedback happens best after evaluation.
comment:8 by , 13 years ago
I think we're all on the same page here. To clarify, not in any way did I want to imply that the patch was added prematurely. Axel saw right through me: I only took it as lame excuse for being too lazy to file a new ticket... :)
comment:9 by , 13 years ago
Yes. The reason it happened that fast was because we were already talking over IRC and I'd forget what was said and done if I'd leave it for later.
Simplify conditional