Mailinglist Archive: opensuse-bugs (5637 mails)
| < Previous | Next > |
[Bug 345506] YaST-GTK: Support 'Image' widget disabling (dim, grey...)
- From: bugzilla_noreply@xxxxxxxxxx
- Date: Tue, 4 Dec 2007 01:37:22 -0700 (MST)
- Message-id: <20071204083722.C91F324538E@xxxxxxxxxxxxxxxxxxxxxx>
https://bugzilla.novell.com/show_bug.cgi?id=345506
User locilka@xxxxxxxxxx added comment
https://bugzilla.novell.com/show_bug.cgi?id=345506#c11
Lukas Ocilka <locilka@xxxxxxxxxx> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC|
|sh@xxxxxxxxxx
AssignedTo|sh@xxxxxxxxxx
|rpmcruz@xxxxxxxxxxxxxxxxxxx
Severity|Normal |Enhancement
Summary|Core, Qt, GTK... New boolean |YaST-GTK:
Support 'Image' widget disabling (dim,
|"HasDimmImageSupport" |grey...)
--- Comment #11 from Lukas Ocilka <locilka@xxxxxxxxxx> 2007-12-04 01:37:22 MST
---
Ricardo, this feature was about dimming an image if it is disabled:
* `Image (... `opt (`disabled) ...)
* UI::ChangeWidget (`id (img_id), `Enabled, false);
UI should be able to dim (turn into gray-scale, darken, ...) an image when it
is disabled and swap it with the original one if enabled again. The
implementation depends on the respective UI. Just the API (widget, widget
manipulation) is the same.
We've decided not to add any other output key to the UI::GetDisplayInfo()
because all UIs that support images can also somehow dim them.
My patch is not going to be used. According to comment #7 we expect that UI
support that. That's also why we don't need to implement any other UI
functions.
In my opinion, we don't need to implement any other `opt (...) parameters such
as `dim or `grey. The reason is simple: UI developers don't need to be bothered
by implementation details. They just want to disable the image and let UI to
deal with it somehow. Internally, the UI can, of course, have several `dim or
`grey options but for UI transparency (across all UIs) they shouldn't be
visible externally.
--- --- ---
So, Ricardo, could you, please, implement the Image widget disabling? In my
opinion all widgets should support it but I've just found that we hadn't needed
it so far (but now we/I do :)...).
--- --- ---
See commits: r42530, r42531, r42532 (Qt3, Qt4: made by Coolo)
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
User locilka@xxxxxxxxxx added comment
https://bugzilla.novell.com/show_bug.cgi?id=345506#c11
Lukas Ocilka <locilka@xxxxxxxxxx> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC|
|sh@xxxxxxxxxx
AssignedTo|sh@xxxxxxxxxx
|rpmcruz@xxxxxxxxxxxxxxxxxxx
Severity|Normal |Enhancement
Summary|Core, Qt, GTK... New boolean |YaST-GTK:
Support 'Image' widget disabling (dim,
|"HasDimmImageSupport" |grey...)
--- Comment #11 from Lukas Ocilka <locilka@xxxxxxxxxx> 2007-12-04 01:37:22 MST
---
Ricardo, this feature was about dimming an image if it is disabled:
* `Image (... `opt (`disabled) ...)
* UI::ChangeWidget (`id (img_id), `Enabled, false);
UI should be able to dim (turn into gray-scale, darken, ...) an image when it
is disabled and swap it with the original one if enabled again. The
implementation depends on the respective UI. Just the API (widget, widget
manipulation) is the same.
We've decided not to add any other output key to the UI::GetDisplayInfo()
because all UIs that support images can also somehow dim them.
My patch is not going to be used. According to comment #7 we expect that UI
support that. That's also why we don't need to implement any other UI
functions.
In my opinion, we don't need to implement any other `opt (...) parameters such
as `dim or `grey. The reason is simple: UI developers don't need to be bothered
by implementation details. They just want to disable the image and let UI to
deal with it somehow. Internally, the UI can, of course, have several `dim or
`grey options but for UI transparency (across all UIs) they shouldn't be
visible externally.
--- --- ---
So, Ricardo, could you, please, implement the Image widget disabling? In my
opinion all widgets should support it but I've just found that we hadn't needed
it so far (but now we/I do :)...).
--- --- ---
See commits: r42530, r42531, r42532 (Qt3, Qt4: made by Coolo)
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
| < Previous | Next > |