Customizer: change title of collapse sidebar anchor when sidebar is collapsed

[thorbrink]

When the sidebar is collapsed, the collapse sidebar anchor should have a different title than "Collapse Sidebar". Perhaps "Expand Sidebar" would make more sense.

[thorbrink]

Screenshot of the bug

[LewisCowles]

Is this in the backend or a theme, as this looks to be a theme-specific issue. More information is needed to proceed.

[thorbrink]

Replying to LewisCowles:

Is this in the backend or a theme, as this looks to be a theme-specific issue. More information is needed to proceed.

This is whn using the Customize panel on any theme. The theme partially visible in the screenshot is Twentyfifteen.
To reproduce: Set theme to Twentyfifteen, go to Customize, click "Collapse sidebar" at the bottom of the Customize panel.

[LewisCowles]

Patch for ./wp-admin/customize.php to reflect more accurately collapse text

[LewisCowles]

admin translation file patched to more accurately describe behaviour

[LewisCowles]

I think this needs to be closed down, two ways to modify the text have been included (the .po translation file being the best way)

[thorbrink]

Replying to LewisCowles:

I think this needs to be closed down, two ways to modify the text have been included (the .po translation file being the best way)

I don't quite follow what you mean by altering the text in the translation file being a good way. This method would leave the original string unaltered, would it not? I have tested the first patch, customizer-toggle.php.patch, and it looks like a good solution and is working for me.

[SergeyBiryukov]

Tickets are closed when a commit is made to WordPress trunk to fix the issue, not when a patch is posted.

[helen]

Replying to LewisCowles:

I think this needs to be closed down, two ways to modify the text have been included (the .po translation file being the best way)

The issue here is about screen readers / text-based navigation and how they explain the interaction to users with sight difficulty or otherwise use accessibility tools. The icon on its own is also quite cryptic, so the title attribute can be helpful even outside of that context. If you're unsure what's happening, feel free to ask in Slack or bypass the ticket. The component and focuses of the ticket should also help to clarify. Let's not confuse new reporters with misunderstandings.

And welcome to Trac, @thorbrink :) Great first bug report!

[LewisCowles]

Guys
@SergeyBiryukov

Tickets are closed when a commit is made to WordPress trunk to fix the issue, not when a patch is posted.

I closed it because it seemed invalid, I was only following what I have watched you do, apologies if this was wrong, but it had nothing to do with the patch

@helen

The issue here is about screen readers / text-based navigation and how they explain the interaction to users with sight difficulty or otherwise use accessibility tools.

Not the issue reported, but okay, then we need more details on a proposed remmedy don't we?

I take on board that the code in-place was confusing and do thank Thor for pointing this out, hence my attempt to provide a middle-ground update via language file & php patch to show Thor how he could modify in either of those two places, neither patch is intended for the core, but AFAIK, I cannot just send Thor attachments; (Happy to learn if I can, but so far there seems to be some problem or another with anything I try to input to WordPress, none so far has given me many clues as to why...)

@thorbrink

I have tested the first patch, customizer-toggle.php.patch, and it looks like a good solution and is working for me.

Thanks for the feedback, apologies for any additional confusion.

Basically WordPress has these language files, where we can change the text, as well as code files, where we can change a lookup string, so I would suggest a language change; rather than a core code change.

I have provided how you could change the text in core file (.php patch), and language file (entire lang file, as I don't know too much about their process, or how that works).

Making the text change when clicked or interacted with would likely require DOM changes, some interactive JS, and over-complicate the text, which was why I decided it might be best to just change the content to a more general language of "Toggle Sidebar" from "Sidebar Collapse" to see if that would be to your mind, clearer.

There are other ways to make this more accessible, but not without changing the DOM code used, and even then I think it may break, so I posted the language file method immediately after posting a subtle change to the code (literally a string literal substitution, which AFAIK, changes the text, at least it did on my 4.1.2)

Thanks all for feedback on my feedback from everyone, I am on #slack if anyone could give guidance on how to make this thing easier.

[helen]

Reports themselves are about problems, not solutions. Requiring that a solution be proposed to understand the problem is not viable.

The string needs to change dynamically in core, in a future release that is not 4.2. We should seek further feedback from the accessibility team as to how the state change is announced, ensuring that focus does not get trapped in that sidebar when it's hidden, etc.

[afercia]

Related: #31201

[celloexpressions]

This was fixed in [33153] (title attribute was removed, text is now dynamic though).