episodes icon indicating copy to clipboard operation
episodes copied to clipboard

Update star and archive icons to be more visible

Open h4iku opened this issue 1 year ago • 7 comments

Hi, thank you for maintaining this app.

This PR improves the visibility of the star, unstar, archive, and unarchive icons. It also deletes the unused resources for star and archive. Fixes #17

main_unarchived main_archived
unstarred_unarchived starred_archived

h4iku avatar Feb 26 '24 05:02 h4iku

I like this. Could you show some pics of the new icons against a lighter white/pale grey background? I wonder what visibility is like in that case.

robin-paulson avatar Feb 27 '24 06:02 robin-paulson

They look a little big on the index but that should fix the issue I had. Looks good to me

ebisbe avatar Feb 27 '24 07:02 ebisbe

Could you show some pics of the new icons against a lighter white/pale grey background? I wonder what visibility is like in that case.

It is somewhat OK. There is no single color that will work with all the backgrounds. The white icons also match the show titles and the three-dot menu on the show page, so they all disappear if the background becomes entirely white.

#C6C6C6 #D3D3D3 #EDEDED
#C6C6C6 #D3D3D3 #EDEDED

Other colors:

yellow orange

They look a little big on the index.

The default size is 32dp.

32dp 28dp 24dp
32dp 28dp 24dp

h4iku avatar Mar 01 '24 02:03 h4iku

True, no single colour will work. A contrasting border, such as black on a white background will be more broadly legible though

robin-paulson avatar Mar 01 '24 03:03 robin-paulson

@robin-paulson

stroked main show

h4iku avatar Mar 01 '24 04:03 h4iku

I'd go with 28dp. I had a similiar issue with the icons and the background and what I did was adding a semitransparent black layer. That with the white or yellow color should work. I like more how the yellow/orange look than with the contrasting border.

ebisbe avatar Mar 01 '24 16:03 ebisbe

The icons with borders are looking good!

robin-paulson avatar Mar 01 '24 19:03 robin-paulson