Page 1 of 1

PR 2227

PostPosted: Wed Feb 11, 2015 19:25
by nrz
Hi,

I want to discuss about PR 2227. The PR seems interesting but i need end user opinion before merge.

Can you review https://github.com/minetest/minetest/pull/2227 ?

I let this discussion opened until 48 hours before merge, but if somes doesn't like, tell me

Re: PR 2227

PostPosted: Wed Feb 11, 2015 19:36
by rubenwardy
It's hardly a big change. Why do you need to consult the community?

Re: PR 2227

PostPosted: Wed Feb 11, 2015 19:37
by Linuxdirk
… and why do you have working translations? :D

Re: PR 2227

PostPosted: Wed Feb 11, 2015 19:38
by indriApollo
He's using gettext I guess. It also works for me (note that I also use the french translation).

On topic: What changes aside from the lighter colors ?

Re: PR 2227

PostPosted: Wed Feb 11, 2015 19:40
by rubenwardy
indriApollo wrote:He's using gettext I guess. It also works for me (note that I also use the french translation).

On topic: What changes aside from the lighter colors ?


None. It literally only changes the color of the list box and the color of the shade around the text box border.

Re: PR 2227

PostPosted: Wed Feb 11, 2015 21:06
by Linuxdirk
indriApollo wrote:He's using gettext I guess. It also works for me (note that I also use the french translation).

Totally does not work for me with any translations. There a re threads in the main section of the forums, in the German sub-section and on Reddit and at least two issues on Github. No-one has a working solution and nowhere I could get a clear statement if it’s broken or if it’s just me. Some say it works, some say it does not work.

But as I learned translations are broken by concept and need to be rewritten completely

rubenwardy wrote:
indriApollo wrote:On topic: What changes aside from the lighter colors ?

None. It literally only changes the color of the list box and the color of the shade around the text box border.

Yeah, let’s stick on-topic here :) To be honest, I actually don’t care for the color change. Both tints look okay to me.

Re: PR 2227

PostPosted: Thu Feb 12, 2015 01:13
by Wuzzy
I am perfectly fine with the presented change.

Re: PR 2227

PostPosted: Thu Feb 12, 2015 06:49
by nrz
Because many users are not on IRC or github, i would talk with users about this graphical change before merge. It seems good for me, but maybe community can talk with us because it's user experience, not only devel.

Re: PR 2227

PostPosted: Thu Feb 12, 2015 09:03
by nrz
Commited

Re: PR 2227

PostPosted: Thu Feb 12, 2015 12:16
by rubenwardy
Small changes aren't worth talking about on forums. It's only a small color change.

If it was a menu redesign, then sure. Or if a tab changes, sure discuss with community.

It is import to involve the community, but I don't think many will notice this change.

Re: PR 2227

PostPosted: Mon Feb 16, 2015 02:04
by philipbenr
I think it is just fine that devs are on the forums. Even if it is a small change.

Re: PR 2227

PostPosted: Mon Feb 16, 2015 18:34
by Krock
Well, that pull was tiny.
It's not required to ask in the forum users for GUI changes like that one everytime.