Opened 11 years ago

Last modified 5 months ago

#1431 in-progress bug

Mail codebase needs a cleanup to fix several issues

Reported by: stippi Owned by: stippi
Priority: normal Milestone: R1
Component: Applications/Mail Version: R1/Development
Keywords: Cc:
Blocked By: #9474 Blocking:
Has a Patch: no Platform: All

Description

  • Mail is not font sensitive and overwrites the user configured system fontsize for some (but not all) controls
  • the code is quite messy and ignores proper object oriented programming concepts
  • the print settings message is used from deleted print jobs instead of being copied
  • the code is riddled with race conditions when it comes to changing settings:
    • program settings are implemented via global variables, in different status of having been cleaned up
    • program settings are applied at "random" with locking problems and race conditions
    • some program settings are available to change in the preference GUI, but have no effect, since the respective code is not in use anymore (for example disabling the text under the icon buttons)

Change History (6)

comment:1 Changed 11 years ago by stippi

Status: newassigned

comment:2 Changed 9 years ago by alexbl

Related to #4495

comment:3 Changed 8 years ago by scottmc

Version: R1/pre-alpha1R1/Development

comment:4 Changed 6 years ago by mmadia

Blocked By: 9474 added

comment:5 Changed 5 months ago by waddlesplash

Most of the mail system now uses the Layout API. Close?

comment:6 Changed 5 months ago by stippi

Using the Layout API was only one bullet point. I don't think the others have been addressed. Please leave open.

Note: See TracTickets for help on using tickets.