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

gui: Make warning titles more readable in Dark Theme #3253

Closed
wants to merge 1 commit into from

Conversation

norgeous
Copy link
Contributor

@norgeous norgeous commented Jun 2, 2016

before:
untitled

after:
untitled

@AudriusButkevicius
Copy link
Member

@st-review merge

@calmh
Copy link
Member

calmh commented Jun 2, 2016

@st-review merge it

@st-review
Copy link

👌 Merged as 8709fec. Thanks, @norgeous!

@st-review st-review closed this Jun 2, 2016
st-review pushed a commit that referenced this pull request Jun 2, 2016
@st-review
Copy link

@calmh: Merge failed:

$ git fetch -f origin refs/pull/3253/head:pr-3253
$ git fetch -f origin master:orig/master
From github.com:syncthing/syncthing
   48245ef..8709fec  master     -> orig/master
$ git reset --hard
HEAD is now at 8709fec gui: Make warning titles more readable in Dark Theme
$ git checkout master
Already on 'master'
Your branch is up-to-date with 'origin/master'.
$ git reset --hard orig/master
HEAD is now at 8709fec gui: Make warning titles more readable in Dark Theme
$ git clean -fxd
Skipping repository syncthing/syncthing
Skipping repository syncthing/syncthing-android
Skipping repository syncthing/docs
Skipping repository syncthing/discosrv
Skipping repository syncthing/syncthing-gtk
Skipping repository syncthing/syncthing-inotify
$ git merge --squash --no-commit pr-3253
Automatic merge went well; stopped before committing as requested
Squash commit -- not updating HEAD
$ git commit -F -
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean

@calmh
Copy link
Member

calmh commented Jun 2, 2016

⚡ :D

norgeous added a commit to norgeous/syncthing that referenced this pull request Jun 2, 2016
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Jun 16, 2017
@syncthing syncthing locked and limited conversation to collaborators Jun 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants