Focus not visible in multiple skins on two controls

Viewing 13 posts - 1 through 13 (of 13 total)
  • Author
    Posts
  • #37872
    HeDiBo
    Participant

    There are several skins where it is hardly discernible whether a TsComboBox or a TsCheckBox has focus.

    This is a project to show that: [attachment=8797:acSetFocus.zip]

    Try to tab to the top TsComboBox or one of the TsCheckBoxes below it. In the default skin (Notes Wood) it is hardly visible that one of these controls has focus. Then switch to a dark skin, like Material Dark, and you will see clearly which control has focus.

    I think all combo boxes when focused should either get a surrounding rectangle (like Cappuccino) or a distinctive color (like Ledenets).

    All check boxes should get a surrounding square when focused. The color change in some skins is attractive but not clear enough.

    #58067
    Support
    Keymaster

    Thank you for a project, I will check it soon.

    #58177
    HeDiBo
    Participant

    In AC 13.17 the combobox now shows a focus rectangle a3.gif

    But the checkboxes only show a faint line at the right hand side. A full focus rectangle was what you intended, I understood. Maybe you can reexamine the source for this.

    There are skins that do not need the focus rectangle. Look at Material Dark, who lights up the surrounding square in a contrasting color. A focus rectangle there would be overkill.

    In other words, maybe the skins need some change.

    #58181
    Support
    Keymaster

    Could you show a screenshot with bad focus rectangle, please?

    #58192
    HeDiBo
    Participant
    'Support' wrote:

    Could you show a screenshot with bad focus rectangle, please?

    If you run the acSetFocus project, I included, after some tabbing you will end up with this: [attachment=8838:FocusCheckBox.jpg]

    Try to find where the focus is.

    The problem is that check boxes without a caption are not highlighted.

    The skin should be changed, to highlight the check box itself, not its possible caption.

    #58197
    Support
    Keymaster

    I'm not sure checkbox will have a good look in all skins in this case (if glyph is in rectangle).

    Standard checkbox doesn't shows a focus rectangle at all. Maybe just do not show it if caption is empty?

    I was wrong, standard checkbox shows same small rectangle if caption is empty.

    #58198
    HeDiBo
    Participant
    'Support' wrote:

    Maybe just do not show it if caption is empty?

    Absolutely not. That's the worst method.

    Have a look at the way check boxes are highlighted in Material Dark skin. You should invent such highlight for other skins too, I think.

    #58199
    Support
    Keymaster

    I see what you mean, but 100+ skins must be repainted for this feature.

    Standard checkboxes shows same small rectangle.

    #58201
    HeDiBo
    Participant
    'Support' wrote:

    Standard checkboxes shows same small rectangle.

    That's why we choose AlphaControls.

    'Support' wrote:

    I see what you mean, but 100+ skins must be repainted for this feature.

    This is a list of skins that are already OK:

    Air, AlterMetro, AlterMetro2016, AlterMetro2017, Aluminium, AutumnSky (not very pronounced though), Bejing, Bejing Ext, Black Box, Blue Shadows (a bit vague), BlueGauze (also a bit vague), Blue Glass, Blue Ice, Canvas Dark (CS), Capuccino, Clean Card (somewhat vague), Clear Sky (CS), Cold (bit vague), DarkMetro, Deep (bit vague), DeepPurple, etc. (I stopped checking because so many are already OK).

    So, not 100+ skins to change…

    #58249
    HeDiBo
    Participant

    No change in 13.18

    #58392
    HeDiBo
    Participant
    'HeDiBo' wrote:

    No change in 13.18

    No change in 14.01 mellow.gif

    #58398
    Support
    Keymaster

    Some skins have a changed checkbox already, some skins – not.

    More time is needed.

    #58421
    HeDiBo
    Participant
    'Support' wrote:

    Some skins have a changed checkbox already, some skins – not.

    More time is needed.

    Not much improvement in 14.02

Viewing 13 posts - 1 through 13 (of 13 total)
  • You must be logged in to reply to this topic.