backdrop-contrib / gin

Admin theme with a strong focus on improving the Editorial Experience.
https://backdropcms.org/project/gin
GNU General Public License v2.0
5 stars 2 forks source link

Image Library needs some love #101

Closed laryn closed 7 months ago

laryn commented 8 months ago

While looking at https://github.com/backdrop-contrib/gin/issues/95 I see that the image library browser needs some love. See screenshots in that issue, and also noting that dark mode also has some core CSS poking through.

EDIT: Changes made on this issue include:

laryn commented 8 months ago

I made a first pass at some improvements here, not yet including the pager (which has it's own issue linked above):

CleanShot 2024-01-10 at 16 57 32@2x

stpaultim commented 8 months ago

I am going to reopen this issue, because I think this is related. After updating GIN on my test site, I'm seeing this CSS poking through with a border that shouldn't be there.

image

@laryn

I don't remember seeing this before.

olafgrabienski commented 8 months ago

@stpaultim Do you mean the blue outline? This is only visible when the field has focus. I see this also with Gin 1.x-3.0.0-alpha8.

laryn commented 8 months ago

That does look like the :focus outline, which is important for accessibility so keyboard navigators know where they are on the page.

stpaultim commented 8 months ago

I'll do some additional testing. I didn't notice this before and suddenly it was very obvious.

laryn commented 7 months ago

I made a minor adjustment here, and closing for now -- reopen if I've missed something. CleanShot 2024-02-16 at 11 18 17@2x