| Summary: | Switching layouts in kdesktop_lock doesn't work | ||
|---|---|---|---|
| Product: | kdesktop | Reporter: | Omri Mezrich <omezrich> |
| Component: | general | Assignee: | Andriy Rysin <arysin> |
| Status: | CLOSED DUPLICATE | ||
| Severity: | normal | CC: | faure, finex, ossi |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Attachments: |
Fix to lockdlg.cc - patched against latest 3.4 branch
Fix to lockdlg.h - patched against latest 3.4 branch |
||
Created attachment 11583 [details]
Fix to lockdlg.cc - patched against latest 3.4 branch
Created attachment 11584 [details]
Fix to lockdlg.h - patched against latest 3.4 branch
Andy Rysin: this seems to be your code. Or Ossi can have a look, maybe. I really don't have a way to test it all, as I don't have an environment for compiling and running this as I haven't been doing active KDE development for more than a year now. I'll try to take a look at the code but without testing not sure how useful it'd be. Andriy --- David Faure <faure@kde.org> wrote: [bugs.kde.org quoted mail] ____________________________________________________ Yahoo! Sports Rekindle the Rivalries. Sign up for Fantasy Football http://football.fantasysports.yahoo.com i will have a look ... *** This bug has been marked as a duplicate of 102149 *** Bug closed. Kdesktop is no more mantained. |
Version: (using KDE Devel) Installed from: Compiled sources Compiler: gcc version 3.3.5 (Debian 1:3.3.5-8ubuntu2) OS: Linux When clicking on the layout switcher in kdesktop_lock, it changes to "err". From what I traced, the problem is in PasswordDlg::layoutClicked: QStringList::iterator it = layoutsList.find( mLayoutButton->text() ); if( it == layoutsList.end() ) { // huh? setLayoutText( "err" ); mLayoutButton->text() contains an ampersand due to KCheckAccelerators adding an accel key. (That's something I don't understand - every button on every dialog becomes automatically accelerated?) anyway - patch attached which just keeps an iterator to the current layout in layoutsList.