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

[.5][happyconcepts] Display account name in lower case letters #358

Closed
btswolf opened this issue Sep 3, 2017 · 15 comments
Closed

[.5][happyconcepts] Display account name in lower case letters #358

btswolf opened this issue Sep 3, 2017 · 15 comments
Labels
[3] Feature Classification indicating the addition of novel functionality to the design
Milestone

Comments

@btswolf
Copy link

btswolf commented Sep 3, 2017

To avoid confusion with the account names I suggest to display the account name with lower case letters consistently in the whole GUI.
image

@ghost
Copy link

ghost commented Sep 3, 2017

The standard reference throughout the GUI:

all "ASSETS" should be "UPPER CASE"
all "user names" should be "lower case"

CAPITALISATION of Tabs/titles should also be removed for visual consistency.

@btsfav
Copy link

btsfav commented Sep 3, 2017

merge with #187 please @wmbutler

@happyconcepts
Copy link
Contributor

happyconcepts commented Sep 3, 2017

@wmbutler @btsfav I can fix the account names for the current sprint if you like; however the relation to #187 eludes me.

EDIT: Confirmed: the recipient's Bitshares address (user input) is already lower-cased in the form field on the SEND page.

happyconcepts added a commit to happyconcepts/bitshares-ui that referenced this issue Sep 4, 2017
@wmbutler
Copy link
Contributor

wmbutler commented Sep 4, 2017

I like the consistency of all assets UC and all uernames LC. Does commit 6b65bb1 cover thi throughout the entire wallet?

@wmbutler wmbutler changed the title Display account name in lower case letters [.5][happyconcepts] Display account name in lower case letters Sep 4, 2017
@wmbutler wmbutler added this to the 170914 milestone Sep 4, 2017
@wmbutler wmbutler added the [3] Feature Classification indicating the addition of novel functionality to the design label Sep 4, 2017
@happyconcepts
Copy link
Contributor

More testing is always welcomed as I don't have much experience writing unit tests. I'm learning Travis but not at production level yet.

@happyconcepts
Copy link
Contributor

FWIW the uppercase transform was being applied to everything in the header; I don't see a use case for capitalizing the account names in any other place in the current GUI.

@wmbutler
Copy link
Contributor

wmbutler commented Sep 4, 2017

It's not a matter of user case really, it's just a consistency thing.

@happyconcepts
Copy link
Contributor

Well in the meantime, if we are sure that React's use of the variable declaration
account_display_name is enforced throughout all the jsx files ... then I can search the codebase:

https://github.com/bitshares/bitshares-ui/search?utf8=%E2%9C%93&q=account_display_name&type=

Is there a 2.0 or 2.x style/implementation guide I can refer to?

@wmbutler
Copy link
Contributor

wmbutler commented Sep 4, 2017

@svk31 may have an answer for you. Should I add a bit more time to accommodate this?

@happyconcepts
Copy link
Contributor

Sure, very happy to help with these smaller issues for 0914.

@svk31 svk31 closed this as completed in 96c5799 Sep 5, 2017
@landry314
Copy link

landry314 commented Oct 13, 2017

If you click on the username tab on the upper right while multiple accounts are starred then a menu pulls down with the usernames still in ALL CAPS. If we are trying to be consistent throughout the GUI then should we change it here too.

@landry314
Copy link

Is commenting in a closed issue thread a bad idea? It seems to get lost more often. I commented here because it is directly related to this issue. There are still account names in uppercase letters...

@happyconcepts, do you want to handle this?

@happyconcepts
Copy link
Contributor

happyconcepts commented Oct 16, 2017 via email

@landry314
Copy link

I am just trying to help the program, not insulting or stalking you. I noticed there is another place where this comes up so I posted it but it seemed to go unnoticed. I think posting in closed issues doesn't work very well so I tagged you because you were assigned this before.

@happyconcepts
Copy link
Contributor

PR is submitted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[3] Feature Classification indicating the addition of novel functionality to the design
Projects
None yet
Development

No branches or pull requests

5 participants