[dreamwidth/dw-free] b4be9b: Remove unused click handler
Branch: refs/heads/master Home: https://github.com/dreamwidth/dw-free Commit: b4be9b0f666ac266c2b8c1ed8589d6bded8ec88f https://github.com/dreamwidth/dw-free/commit/b4be9b0f666ac266c2b8c1ed8589d6bded8ec88f Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2021-02-05 (Fri, 05 Feb 2021)
Changed paths: M htdocs/js/components/jquery.icon-browser.js
Log Message:
Remove unused click handler
Now that it's a one-click interaction, the select button is gone.
Commit: 210d5dc9b4708ec2d717a801696170fc8d926240 https://github.com/dreamwidth/dw-free/commit/210d5dc9b4708ec2d717a801696170fc8d926240 Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2021-02-05 (Fri, 05 Feb 2021)
Changed paths: M htdocs/js/components/jquery.icon-browser.js
Log Message:
Remove redundant click trigger on enter key
This was an accessibility workaround for when we were misusing <a>
elements
instead of using proper <button>
s. Now that we get the default accessible
button behavior, this is cruft.
Commit: cdb5842f57f1bf547bd888144abfce3fb4e2e9cb https://github.com/dreamwidth/dw-free/commit/cdb5842f57f1bf547bd888144abfce3fb4e2e9cb Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2021-02-05 (Fri, 05 Feb 2021)
Changed paths: M htdocs/js/components/jquery.icon-browser.js M htdocs/scss/components/icon-browser.scss M views/components/icon-browser.tt
Log Message:
Add show/hide button for icon browser options on mobile
The display options take up way too much space on mobile, and presumably you like what you like and aren't changing them every time!
So at small viewport sizes, this commit hides the options behind a toggle button.
It goes right above+before the search field. If you summon the browser without an icon already selected, the search field is the first thing focused, so you don't have to tab through the options toggle before reaching the icons if you're keyboard-navvin.
For sighted users: The button is styled to look like a cut tag or a
<details>
element (with the little open/close disclosure triangle), which DW users should be intimately familiar with. Its text includes a summary of the currently selected options.For screen readers: I'm using an
aria-label
instead of the current options summary, because they won't have the visual cue of the disclosure triangle.Commit: ca17b1420222b10563ca82367a955a0ccff89f9f https://github.com/dreamwidth/dw-free/commit/ca17b1420222b10563ca82367a955a0ccff89f9f Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2021-02-05 (Fri, 05 Feb 2021)
Changed paths: M htdocs/js/components/jquery.icon-browser.js M htdocs/js/components/jquery.icon-select.js M views/components/icon-select-dropdown.tt M views/components/icon-select-icon.tt M views/journal/quickreply.tt M views/journal/talkform.tt
Log Message:
Restore missing behavior: focus comment body on dismissal
This went missing after #2840.
Commit: de017f8fb389a476ab309d957cd7c4a7c8dc25bd https://github.com/dreamwidth/dw-free/commit/de017f8fb389a476ab309d957cd7c4a7c8dc25bd Author: Nick Fagerlund nick.fagerlund@gmail.com Date: 2021-02-05 (Fri, 05 Feb 2021)
Changed paths: M views/components/icon-browser.tt
Log Message:
Use same inline styles for disclosure triangle that we use in cut tags
Compare: https://github.com/dreamwidth/dw-free/compare/2b83bc55f687...de017f8fb389