You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the pad Settings dialog, checking Disable chat disables the underlying behaviour but does not visually disable the dependent Chat always on screen control. The checkbox still looks enabled / clickable to the user, which is misleading UX.
Reproduction
Open a pad.
Open Settings (toolbar cog icon).
Under "My View": tick the Disable chat checkbox.
Note that Chat always on screen is functionally disabled, but visually renders as if enabled (no greyed-out label, checkbox still looks interactive).
Expected
When Disable chat is checked, the dependent "Chat always on screen" row should be visually disabled: greyed label, disabled attribute on the checkbox, etc.
Small fix: toggle disabled attribute on #options-chatandusers in the #options-disablechat change handler, or wrap both in a fieldset that toggles disabled collectively.
In the pad Settings dialog, checking Disable chat disables the underlying behaviour but does not visually disable the dependent Chat always on screen control. The checkbox still looks enabled / clickable to the user, which is misleading UX.
Reproduction
Expected
When Disable chat is checked, the dependent "Chat always on screen" row should be visually disabled: greyed label,
disabledattribute on the checkbox, etc.Notes
disabledattribute on#options-chatandusersin the#options-disablechatchange handler, or wrap both in a fieldset that togglesdisabledcollectively.Discovered while manual-testing #7584.