search

jecisc / Bazard

0 stars 0 forks source link

GLMTabLabelBrick #selected: seems to not do what we think he does #236

Closed jecisc closed 5 years ago

jecisc commented 5 years ago

Migrated case from Manuscript.

Original case: https://pharo.fogbugz.com/f/cases/18269 Status: Work Needed Project: GTools Original Author: CyrilFerlicot Date: 18 May 2016 9:59:50 pm

Description:

Maybe this is something wanted be I think that this method is not right:

selected: aBoolean "Use the appropriate label."

self isSelected
    ifTrue: [ self beSelected ]
    ifFalse: [ self beNormal ]

It take a parameter but does not use it. Shouldn't it be "aBoolean" instead of "self isSelected"?

jecisc commented 5 years ago

Reply: Author: Pavel Krivanek Date: 27 January 2017 12:21:28 pm

Message:

Low priority issue. Not for Pharo 6. The milestone "Later" was set automatically.