[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-usability
Subject:    Re: Please extend KXMLGUI to integrate RMB menus (was Re: Konqueror
From:       Waldo Bastian <bastian () kde ! org>
Date:       2003-07-23 12:20:55
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 23 July 2003 13:37, Henrique Pinto wrote:
> On Wednesday 23 July 2003 05:54, Waldo Bastian wrote:
> > If you tell me which exact code (file, line-numbers) you are talking
> > about I will be better able to tell you how and if the KIOSK-related code
> > should be handled.
>
> kdebase/kdesktop/krootwm.cc, lines 68-198 and 237-452

The actions that are being made conditionally shouldn't be a problem. If the 
action doesn't exist XMLGUI will not do anything with it either. So I think 
the part 68-198 should be mostly left as is.

Most of 237-452 involves checking if an action exist and then plugging it in. 
XMLGUI will do this checking automatically. It will never plug an action that 
doesn't exist. You may run into problems with sub-menus that do not have any 
actions in them. I don't know if XMLGUI is smart enough to hide the whole 
submenu in such case.

If you can manage to leave 68-198 mostly untouched (I don't think you need to 
change anything there, do you?) then you should be able to basically replace 
237-452 with the XMLGUI stuff. At that point I guess we need to look how well 
XMLGUI behaves with submenus and such.

Cheers,
Waldo
- -- 
bastian@kde.org -=|[ SuSE, The Linux Desktop Experts ]|=- bastian@suse.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE/Hn2nN4pvrENfboIRAmTJAKCnl8eFmuqDSkj40ffu1e7u9F7rWwCgj7DR
CqZrnOQGJ+QbjP30AC8zrSE=
=pWij
-----END PGP SIGNATURE-----
_______________________________________________
kde-usability mailing list
kde-usability@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-usability

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic