Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use modal-mask--dark selector to make icons light + add light ⬇️ dowload icon #1059

Merged
merged 1 commit into from
Nov 4, 2021

Conversation

beardhatcode
Copy link
Contributor

@beardhatcode beardhatcode commented Nov 4, 2021

When using a share link the download icon is rendered black instread of white

before:
screenshot_2021-11-04-102732

after
screenshot_2021-11-04-102809

As of the following commit, we may use modal-mask--dark as selector to know if the modal is dark.
nextcloud-libraries/nextcloud-vue@0cd2258

Steps to reproduce:
  1. create a folder with some pictures in it
  2. create a read-only share link
  3. open te link
  4. open a picture

…load icon

As of the following commit, we may use modal-mask--dark as selector to
know if the modal is dark.
nextcloud-libraries/nextcloud-vue@0cd2258

Signed-off-by: Robbert Gurdeep Singh <git@beardhatcode.be>
@beardhatcode beardhatcode merged commit c6efba1 into master Nov 4, 2021
@beardhatcode beardhatcode deleted the fix/single-action-icons branch November 4, 2021 09:44
@skjnldsv skjnldsv mentioned this pull request Nov 8, 2021
23 tasks
@szaimen
Copy link
Contributor

szaimen commented Nov 15, 2021

/backport to stable22

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Nov 15, 2021
@szaimen
Copy link
Contributor

szaimen commented Nov 15, 2021

/backport to stable21

@backportbot-nextcloud
Copy link

The backport to stable21 failed. Please do this backport manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants