UI enhancement Success

User tests: Successful: Unsuccessful:

avatar allenzhao
allenzhao
28 Aug 2014

Remove color display in category drop down and order by category title.

avatar allenzhao allenzhao - open - 28 Aug 2014
avatar elkuku
elkuku - comment - 28 Aug 2014

Maybe we can still have our colors if we do a

data-content="<span class='label' style='background-color: #{{ category.color }};'>&nbsp;&nbsp;</span> {{ category.title }}"

:wink:

avatar b2z
b2z - comment - 28 Aug 2014

Exactly, like on GH.

avatar elkuku elkuku - change - 28 Aug 2014
Labels Added: UI
avatar allenzhao
allenzhao - comment - 28 Aug 2014

screenshot 2014-08-29 00 23 01
Should be better than 'ugly'

avatar allenzhao
allenzhao - comment - 28 Aug 2014

By the way, I'm not so sure about this, because I didn't write the style thing, as you can see it's the same with GitHub's displaying color, as I just copied it's style attributes.
Is this OK or we can't do that?

avatar allenzhao
allenzhao - comment - 28 Aug 2014

Oh just went through GitHub's terms of service

The look and feel of the Service is copyright ©2010 GitHub Inc. All rights reserved. You may not duplicate, copy, or reuse any portion of the HTML/CSS, Javascript, or visual design elements or concepts without express written permission from GitHub.

Sad it's not open source itself.

avatar allenzhao
allenzhao - comment - 28 Aug 2014

Reverted the change.

data-content="<span class='label' style='background-color: #{{ category.color }};'>&nbsp;&nbsp;</span> {{ category.title }}"

Does not look so good. So is there a better way than this? Not familiar with the CSS stuff...

avatar b2z
b2z - comment - 28 Aug 2014

Wow! That's really sad :(

avatar allenzhao
allenzhao - comment - 28 Aug 2014

I thought it would be an open source software as it's hosting so many FOSS on it.
Sadly it's not ;( And we have to come up with our way of displaying color, otherwise we have to abandon the color stuff. But to be honest, I like displaying color...

avatar elkuku
elkuku - comment - 28 Aug 2014

Sorry I'm not getting it.. The screen shot you posted looked really good.

I don't think that we will get into any license trouble with this code, especially with GitHub :wink:

avatar allenzhao
allenzhao - comment - 28 Aug 2014

The style attribute of the srceenshot is the same as GitHub as I just simply copied from its CSS file. 

I thought GitHub should be open source and we are okay to just copy its design. But the terms of service says another thing...

Sent from Mailbox

On Fri, Aug 29, 2014 at 12:45 AM, Nikolai Plath notifications@github.com
wrote:

Sorry I'm not getting it.. The screen shot you posted looked really good.

I don't think that we will get into any license trouble with this code, especially with GitHub :wink:

Reply to this email directly or view it on GitHub:
#457 (comment)

avatar elkuku
elkuku - comment - 28 Aug 2014

Ah well, just leave off the CSS stuff then - labels already have rounded corners in Bootstrap, let those UI experts do the "fine tuning" IF they agree that this one is less ugly :wink:3

But to be on the safe side, you would have to somehow "hard revert" this 4d401df commit, because the code already "exists" - but I'm not a license expert either...

avatar elkuku
elkuku - comment - 28 Aug 2014

Does not look so good.

debug-3882

It looks like this without any CSS changes - ugly?

avatar b2z
b2z - comment - 28 Aug 2014

It looks like this without any CSS changes - ugly?

ok ;)

avatar allenzhao
allenzhao - comment - 28 Aug 2014

@elkuku On my laptop the square looks more like a triangle like this:
screenshot 2014-08-29 01 09 40

avatar allenzhao
allenzhao - comment - 28 Aug 2014

And BTW, how to hard revert the commits?
I went through this, but I think it's how to remove a file?
If I rebase and delete this branch, will the commit still be there?

avatar allenzhao
allenzhao - comment - 28 Aug 2014

Found a solution here, and will do it later.
http://blog.stevenocchipinti.com/2014/03/10/completely-removing-a-git-commit-due-to-sensitive-information/
It's like I've done something illegal and trying to do some dangerous stuff to cover it up..

avatar elkuku
elkuku - comment - 28 Aug 2014

I am sure one of our CSS gurus will find some nifty lines to make it look a perfect square - to me it's "not that bad" :D

how to hard revert the commits?

You could simply open a new PR from master and apply the diff from here - no problem ;)

avatar b2z
b2z - comment - 29 Aug 2014

I am sure one of our CSS gurus will find some nifty lines to make it look a perfect square - to me it's "not that bad" :D

Yeap, so @allenzhao do not bother so much ;)

avatar b2z
b2z - comment - 31 Aug 2014

@allenzhao will you make a new PR?

avatar allenzhao
allenzhao - comment - 31 Aug 2014

Sure, maybe later today.

Cheers | 顺颂时祺
Allen Zhao | 赵泽涵

在 2014年8月31日,14:48,Dmitry Rekun notifications@github.com 写道:

@allenzhao will you make a new PR?


Reply to this email directly or view it on GitHub.

avatar allenzhao allenzhao - close - 31 Aug 2014
avatar allenzhao allenzhao - close - 31 Aug 2014
avatar allenzhao allenzhao - change - 31 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-31 11:43:44
avatar allenzhao allenzhao - head_ref_deleted - 31 Aug 2014

Add a Comment

Login with GitHub to post a comment