Skip to content

NA - #31904

Closed
AndyXheli wants to merge 8 commits into
nextcloud:masterfrom
AndyXheli:patch-2
Closed

NA#31904
AndyXheli wants to merge 8 commits into
nextcloud:masterfrom
AndyXheli:patch-2

Conversation

@AndyXheli

Copy link
Copy Markdown
Contributor

Signed-off-by: Andrea Xheli axheli@axtsolutions.com

Should fix LDAP AD Integration Dark Theme issue #25809

Updated code Ref Fix LDAP Dark Theme Issue #26666

Before
image

After Dark Theme
image

After Normal Theme
image

Signed-off-by: Andrea Xheli axheli@axtsolutions.com
Should fix LDAP AD Integration Dark Theme issue #25809
Updated code Ref Fix LDAP Dark Theme Issue #26666

@CarlSchwanCarlSchwan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one :)

@AndyXheli

Copy link
Copy Markdown
ContributorAuthor

Thank you very much!! :)

Comment threadapps/user_ldap/css/settings.css Outdated
@szaimenszaimen added bug 3. to review Waiting for reviews labels Apr 8, 2022
@szaimenszaimen added this to the Nextcloud 24 milestone Apr 8, 2022
Signed-off-by: Andrea Xheli axheli@axtsolutions.com
@AndyXheli

Copy link
Copy Markdown
ContributorAuthor

Just updated it with them removed.

@AndyXheli

AndyXheli commented Apr 8, 2022

Copy link
Copy Markdown
ContributorAuthor

Should we also backport this ?

Comment threadapps/user_ldap/css/settings.css Outdated
Signed-off-by: Andrea Xheli axheli@axtsolutions.com
@szaimen

Copy link
Copy Markdown
Contributor

@AndyXheli can you please fix DCO and squash your commits? Afterwards we can merge :)

Andy Xheliand others added 5 commits April 8, 2022 11:16
Signed-off-by: Andrea Xheli axheli@axtsolutions.com
Should fix LDAP AD Integration Dark Theme issue #25809
Updated code Ref Fix LDAP Dark Theme Issue #26666
Signed-off-by: Andy Xheli <axheli@SWCD.CAD.SYSTEM>
Signed-off-by: Andrea Xheli axheli@axtsolutions.com
Signed-off-by: Andy Xheli <axheli@SWCD.CAD.SYSTEM>
Signed-off-by: Andrea Xheli axheli@axtsolutions.com
Signed-off-by: Andy Xheli <axheli@SWCD.CAD.SYSTEM>
@AndyXheli

Copy link
Copy Markdown
ContributorAuthor

@szaimen I'm sorry can you please help me. I have no idea what I did. I apologies :(

@szaimen

Copy link
Copy Markdown
Contributor

@szaimen I'm sorry can you please help me. I have no idea what I did. I apologies :(

In this case just open a new PR with this bugfix in a single commit and don't forget the sign-off ;)

@AndyXheli
AndyXheli deleted the patch-2 branch April 8, 2022 16:46
@AndyXheliAndyXheli changed the title Fix LDAP Dark Theme IssueNAApr 8, 2022
@szaimenszaimen removed this from the Nextcloud 24 milestone Apr 8, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@AndyXheli@szaimen@CarlSchwan