abaevbog
abaevbog
> The one thing I would want to check is whether we're assign a proper role (ARIA alert role?) to the save popup, so that it gets announced by screen...
It comes from [here](https://github.com/zotero/zotero/blob/38391f356c181d4e561c9cddf443d795ce92fac0/chrome/skin/default/zotero/integration.css#L257) which was set 13 years ago, so my understanding is that it has always been that way. It applies to macOS the same way as to...
So I dropped the opacity and there is something to note here. On windows, I noticed that with `opacity: 0.9` removed, there is a shadow appearing below and to the...
@yexingsha AA is a minimum that we need to hit so `#0000008c` as `fill-secondary` works just fine, in my opinion. This actually seems to clear all the contrast warnings that...
@mrtcode so a couple of issues here that I did not touch in https://github.com/zotero/reader/pull/127 because they are a bit trickier are vpat 75 and 78. I thought I should check...
I don't believe there are any strict requirements as to where in the DOM they need to be. Overall, any node can be marked as `aria-live` and screen readers should...
@mrtcode Apologies for late reply! I was moving last weekend and had to drive from one coast of the United States to another... > since it’s not needed for EPUB...
> That sounds right to me. You can merge your changes, and I’ll make a few tweaks. Cool! I just did with a minor cleanup and a bit more helpful...
Awesome! Hopefully (fingers crossed), with this we will have all the most pertinent reader issues covered!
It makes sense if the focus is initially in the itemTree. What about if the focus is elsewhere? I have these two suggestions, though I'm not sure about either of...