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

Fix illegible text color in columns Size and Modified on dark theme #11823

Conversation

lex111
Copy link
Contributor

@lex111 lex111 commented Oct 14, 2018

Resolves #11441

image

@MorrisJobke MorrisJobke added design Design, UI, UX, etc. 3. to review Waiting for reviews labels Oct 15, 2018
@MorrisJobke MorrisJobke added this to the Nextcloud 15 milestone Oct 15, 2018
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Very simple, I really like it :)

Signed-off-by: Alexey Pyltsyn <lex61rus@gmail.com>
@lex111 lex111 force-pushed the fix-illegible-text-color-in-size-and-modified-columns branch 2 times, most recently from a0bcac1 to 66d2333 Compare October 16, 2018 18:45
@lex111
Copy link
Contributor Author

lex111 commented Oct 17, 2018

Hi guys, please watch this PR 😃

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Looks good to me, great work @lex111!

@jancborchardt
Copy link
Member

@juliushaertl can you check if your technical feedback was fixed? :)

@juliusknorr
Copy link
Member

Yes, fine by me now. Does this also work for you @skjnldsv ?

@juliusknorr
Copy link
Member

@lex111 Can you add a signoff message to your commits as described here https://github.com/nextcloud/server/blob/master/.github/CONTRIBUTING.md#sign-your-work

@lex111 lex111 force-pushed the fix-illegible-text-color-in-size-and-modified-columns branch 2 times, most recently from 70834d5 to 968a63a Compare October 18, 2018 17:44
…rework after review

Signed-off-by: Alexey Pyltsyn <lex61rus@gmail.com>
@lex111 lex111 force-pushed the fix-illegible-text-color-in-size-and-modified-columns branch from 968a63a to 30cbd13 Compare October 18, 2018 17:50
@lex111
Copy link
Contributor Author

lex111 commented Oct 18, 2018

@juliushaertl done ✔️

@lex111
Copy link
Contributor Author

lex111 commented Oct 19, 2018

friendly ping @denismosolov I'm not late? 😄

@skjnldsv
Copy link
Member

Yes, fine by me now. Does this also work for you @skjnldsv ?

Good for me ;)

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 19, 2018
@MorrisJobke MorrisJobke merged commit 6a979a0 into nextcloud:master Oct 19, 2018
@MorrisJobke
Copy link
Member

Good for me ;)

Then press also the green button ;)

@denismosolov
Copy link
Contributor

denismosolov commented Oct 19, 2018

friendly ping @denismosolov I'm not late?

You rock!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish design Design, UI, UX, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants