[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