« Return to Thread: Ctrl+L
Re: Ctrl+L
by Bugzilla from thiago@kde.org
::
Rate this Message:
Nick Shaforostoff wrote:
>Please review the following patch.
>It makes Konqueror behaive on Ctrl+L like the rest: just select current
> text in address bar, so that it can be easily copied or erased by
> typing (pasting) new one.
>
>currently it removes the previous text and there is no way to have it
> back (i.e. ctrl+z doesn't work)
>
>also currently m_combo->clearTemporary(); is called two times during
> initialization (directly and via slotClearLocationBar), and the patch
> fixes this.KDE 3's Konqueror had a "Location" label next to the line edit, which had
an accelerator. So you could do Alt+o to select it, without erasing the
text.
So, if the label were still there, I'd oppose the patch as unnecessary.
Now, I agree with it.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
>Please review the following patch.
>It makes Konqueror behaive on Ctrl+L like the rest: just select current
> text in address bar, so that it can be easily copied or erased by
> typing (pasting) new one.
>
>currently it removes the previous text and there is no way to have it
> back (i.e. ctrl+z doesn't work)
>
>also currently m_combo->clearTemporary(); is called two times during
> initialization (directly and via slotClearLocationBar), and the patch
> fixes this.
an accelerator. So you could do Alt+o to select it, without erasing the
text.
So, if the label were still there, I'd oppose the patch as unnecessary.
Now, I agree with it.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
signature.asc (196 bytes) « Return to Thread: Ctrl+L
| Free embeddable forum powered by Nabble | Forum Help |
