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

2FA settings icons don’t invert in dark theme #13643

Closed
jancborchardt opened this issue Jan 17, 2019 · 1 comment
Closed

2FA settings icons don’t invert in dark theme #13643

jancborchardt opened this issue Jan 17, 2019 · 1 comment
Labels
1. to develop Accepted and waiting to be taken care of bug design Design, UI, UX, etc. feature: settings good first issue Small tasks with clear documentation about how and in which place you need to fix things in.

Comments

@jancborchardt
Copy link
Member

With the dark theme, 2FA settings look like this:
dark theme 2fa icons

That’s because they are loaded in the <img> tag. Ideally it should be changed to use a class on the header like icon-… so it’s used as background-image.

Documentation on that at: https://docs.nextcloud.com/server/latest/developer_manual/design/css.html#scss-icons-mixins

@jancborchardt jancborchardt added bug 1. to develop Accepted and waiting to be taken care of feature: settings labels Jan 17, 2019
@jancborchardt jancborchardt removed their assignment Jan 28, 2019
@jancborchardt jancborchardt added good first issue Small tasks with clear documentation about how and in which place you need to fix things in. design Design, UI, UX, etc. labels Jan 28, 2019
@nickvergessen
Copy link
Member

Code in question

/** @var \OCP\Authentication\TwoFactorAuth\IProvidesPersonalSettings $provider */
$provider = $data['provider'];
if ($provider instanceof \OCP\Authentication\TwoFactorAuth\IProvidesIcons) {
$icon = $provider->getDarkIcon();
} else {
$icon = image_path('core', 'actions/password.svg');
}

This was referenced Mar 4, 2019
GretaD added a commit that referenced this issue May 17, 2019
GretaD added a commit that referenced this issue May 21, 2019
GretaD added a commit that referenced this issue May 22, 2019
skjnldsv added a commit that referenced this issue May 29, 2019
Start fixing 2fa icons in dark theme, ref #13643
nachoparker pushed a commit to nachoparker/server that referenced this issue Jun 28, 2019
@MorrisJobke MorrisJobke mentioned this issue Jul 15, 2019
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug design Design, UI, UX, etc. feature: settings good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Development

No branches or pull requests

3 participants