Skip to content

Give audio-white and audio-off-white shadow via CSS#501

Merged
nickvergessen merged 1 commit into
masterfrom
give-audio-icons-shadow
Nov 22, 2017
Merged

Give audio-white and audio-off-white shadow via CSS#501
nickvergessen merged 1 commit into
masterfrom
give-audio-icons-shadow

Conversation

@pixelipo

@pixelipo pixelipo commented Nov 22, 2017

Copy link
Copy Markdown

Fixes regression nextcloud/server#7243

audio-white and audio-off-white should be visible in Talk app when video is disabled (thanks to shadow).

This introduces new class .icon-shadow that can be use on any icon. Other icons that (still) have SVG shadow filters will be migrated to this solution in a subsequent PR, because only this one is critical.

To be reviewed and merged with nextcloud/server#7247

Signed-off-by: Marin Treselj <marin.treselj@forlagshuset.no>
@nickvergessen nickvergessen merged commit ff4298b into master Nov 22, 2017
@nickvergessen nickvergessen deleted the give-audio-icons-shadow branch November 22, 2017 14:09
@nickvergessen

nickvergessen commented Nov 22, 2017

Copy link
Copy Markdown
Member

Hmm after looking at the icon for a while, I wonder if the shadow should be darker.
It's still hardly visible.

@pixelipo

Copy link
Copy Markdown
Author

@nickvergessen I'll be making a set of PRs to migrate other shadowed icons (in Gallery app as well) to this solution in the next couple of days. I can propose a stronger shadow then.

I guess we can close nextcloud/server#7243 now

@pixelipo pixelipo mentioned this pull request Nov 28, 2017
marcoambrosini pushed a commit that referenced this pull request Oct 9, 2019
…ttrs-on-datepicker

Allow to bind attributes to the ActionInput timepicker if present
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants