Skip to content

Conversation

Spekular
Copy link
Member

Separate from the earlier background change as this is entirely subjective. (If you'd rather I posted this change in that pull request please tell me).
compare

Darkens the background art to make windows stand out more. Separate from
background fix as this is a matter of aesthetics/subjective.
Fixes odd border on previous art.
@diizy
Copy link
Contributor

diizy commented Sep 28, 2014

We just updated the background art... why do you want to change it again? In the future, please discuss potential changes to UI/graphics on the mailing list or issue tracker before committing pull requests.

@Spekular
Copy link
Member Author

@diizy Ok, sorry. Should I close this? I thought they could be separate since this is more subjective than the other one.

@diizy
Copy link
Contributor

diizy commented Sep 28, 2014

Generally, we discuss potential changes on the mailing list or issue tracker before commiting them as pull requests.

@Spekular
Copy link
Member Author

@diizy Ah ok. I thought issues were "Here's a problem, what do we do to fix it", and pull requests were "I have a solution/change I want to make, do you agree?". So should I close this or leave it?

@Umcaruje
Copy link
Member

I think you should close this.

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