Opened 12 years ago

Last modified 15 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 by stippi, 12 years ago

Status: newassigned

comment:2 by alexbl, 10 years ago

Related to #4495

comment:3 by scottmc, 9 years ago

Version: R1/pre-alpha1R1/Development

comment:4 by mmadia, 7 years ago

Blocked By: 9474 added

comment:5 by waddlesplash, 15 months ago

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

comment:6 by stippi, 15 months ago

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.