[Users] [Bug 3817] New: Rework alertpanel focus handling
noreply at thewildbeast.co.uk
noreply at thewildbeast.co.uk
Thu Apr 27 23:35:29 CEST 2017
http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3817
Bug ID: 3817
Summary: Rework alertpanel focus handling
Classification: Unclassified
Product: Claws Mail
Version: GIT
Hardware: PC
OS: Linux
Status: NEW
Severity: minor
Priority: P3
Component: Other
Assignee: users at lists.claws-mail.org
Reporter: flo.xfce at gmx-topmail.de
Created attachment 1747
-->
http://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1747&action=edit
Rework alertpanel focus handling
The current way of giving focus to a button other than button1 in the
alertpanel we prepend a plus sign to the label. This is bad style. Also, this
breaks building with the --enable-gtk3 flag (which is unsupported atm, true,
but still another reason to rework the current code).
The attached patch introduces a seperate flag to specify which button in the
alertpanel should be focused. A bit more typing is neccessary, but yields much
cleaner code. The patch is rather big because there are so many calls to
alertpanel(), but the changes are trivial.
The patch was created with git diff origin/master..alertpanel_rework -- .
':!po'
--
You are receiving this mail because:
You are the assignee for the bug.
More information about the Users
mailing list