[Bug 345506] New: Core, Qt, GTK... New boolean "HasDimmImageSupport"
https://bugzilla.novell.com/show_bug.cgi?id=345506 Summary: Core, Qt, GTK... New boolean "HasDimmImageSupport" Product: openSUSE 11.0 Version: unspecified Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: YaST2 AssignedTo: sh@novell.com ReportedBy: locilka@novell.com QAContact: jsrain@novell.com CC: coolo@novell.com, tgoettlicher@novell.com Found By: --- Coolo has implemented a new Image functionality to Qt (3,4) to support Image-Dimming this support is still missing in $other UIs (and the future support is unclear) so I've prepared a patch for Core + Qt that adds another boolean to UI::GetDisplayInfo(); Would it be possible to add it there, please? (see the proposed patches) Anyway, if there was a better way, I'd like it too :))) This was the only method I was able to use. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c1 --- Comment #1 from Lukas Ocilka <locilka@novell.com> 2007-12-03 05:15:32 MST --- Created an attachment (id=185609) --> (https://bugzilla.novell.com/attachment.cgi?id=185609) Core -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c2 --- Comment #2 from Lukas Ocilka <locilka@novell.com> 2007-12-03 05:15:57 MST --- Created an attachment (id=185610) --> (https://bugzilla.novell.com/attachment.cgi?id=185610) Qt (3) Might be the same for Qt4... -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c3 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rpmcruz@alunos.dcc.fc.up.pt --- Comment #3 from Lukas Ocilka <locilka@novell.com> 2007-12-03 05:17:45 MST --- Coolo: could you please write more details on the implemented feature? Ricardo might be so kind and implement it also in GTK UI :) -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c4 --- Comment #4 from Stephan Kulow <coolo@novell.com> 2007-12-03 08:54:34 MST --- I talked with Huha about it and the general agreement was that ycp should not cover for this. gtk still needs to implement it though - the thing is YImage::setEnabled(false) should update the displayed pixmap to appear in gray and YImage::setEnabled(true) should reenable the original pixmap. Qt implements it in reloading the pixmap, with a filter applied if it's disabled. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c5 --- Comment #5 from Lukas Ocilka <locilka@novell.com> 2007-12-03 08:59:43 MST --- Er, ... anyway, I need to control the image enabling/disabling or dimming or whatever you call that from YCP :) How do I do that, please? -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c6 --- Comment #6 from Lukas Ocilka <locilka@novell.com> 2007-12-03 09:03:09 MST --- My patch (proposal) was about this: if (UI_supports_image_disabling) { use_image_enabling_disabling_method(); } else { use_another_method(); // for instance replace_point/replace(widget) } but I know that is quite specific and maybe I'm the only one who wants it :) I wanted to have it easy for myself :))) heh I do accept any solution ;) -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c7 --- Comment #7 from Stephan Kulow <coolo@novell.com> 2007-12-03 09:35:54 MST --- just blindly assume UIs with image support will dim if you disable I would say. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c8 --- Comment #8 from Stefan Hundhammer <sh@novell.com> 2007-12-03 09:37:00 MST --- Yes. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506#c9 --- Comment #9 from Lukas Ocilka <locilka@novell.com> 2007-12-03 09:42:11 MST --- Well, and what do do if it doesn't support dimming :)? Just ignore it? OK. In that case, it is "fixed" :) -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506 User rpmcruz@alunos.dcc.fc.up.pt added comment https://bugzilla.novell.com/show_bug.cgi?id=345506#c10 --- Comment #10 from Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> 2007-12-03 13:47:37 MST --- I read this on the mailing list. This is a pretty trivial feature. What about other effects like gray, btw? I won't implement this on yast-gtk because it doesn't even work against recent versions of libyui. But sure, this can be done. We even have code for this from the find entry icons on the help dialog. I'll point that out on a comment on our YImage implementation. Something is wrong about the libyui attachment, right? Or have you just added those fallback hooks for now? Btw, I'd use a more generic name like UI::HasImageEffectsSupport(). Whats important is whether the UI code can do pixel manipulation. Qt and GTK+ can. Wt can't. (needless to say, ncurses doesn't even support images, so it would return false too.) How do you intend to enable this feature from the code? If you add an `opt option at the widget constructor, then of course will have to construct several widgets, which means using ReplacePoints. Maybe we could add a UI::ImageApplyEffect(image, `dim|`grey...) ? The UI could even optimize this by keeping the several transformations in memory, as demanded, until the image is destructed. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=345506#c11 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sh@novell.com AssignedTo|sh@novell.com |rpmcruz@alunos.dcc.fc.up.pt Severity|Normal |Enhancement Summary|Core, Qt, GTK... New boolean |YaST-GTK: Support 'Image' widget disabling (dim, |"HasDimmImageSupport" |grey...) --- Comment #11 from Lukas Ocilka <locilka@novell.com> 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506 User rpmcruz@alunos.dcc.fc.up.pt added comment https://bugzilla.novell.com/show_bug.cgi?id=345506#c12 --- Comment #12 from Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> 2007-12-04 15:05:14 MST --- Right, didn't read the thread carefully. Implemented it. GTK Image widget behaves the same as you say, but we don't use it, as yast-ui needs more stuff than it features, the scaling and tiling. Used its code. Its interesting that GtkImage asks the current Style to render it, so a style could make it slightly blue or whatever. Most will just default though, to gray. -- 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.
https://bugzilla.novell.com/show_bug.cgi?id=345506 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=345506#c13 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #13 from Lukas Ocilka <locilka@novell.com> 2007-12-05 06:29:55 MST --- Let's close it 'FIXED' if 'implemented' :))) Additional features and ideas are just being discussed on yast-devel mailing list ;) -- 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.
participants (1)
-
bugzilla_noreply＠novell.com