New issue
Advanced search Search tips

Issue 854249 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 31
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change how TableViewTextItem sets cell textColor

Project Member Reported by thegreenfrog@chromium.org, Jun 19 2018

Issue description

The if/else conditionals in configureCell: seems too complex and is prone for unexpected behavior. 

Things to consider:
- TextItemColorBlack is essentially 0, so just checking self.textColor when that is the value never hits that conditional. The current implementation of the first if clause is a temp patch for this.
- What is the value of the styler? Should it be the default?
- What is the default color? lightGray? Black?
 
Status: WontFix (was: Assigned)

Sign in to add a comment