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

New UI hover effects #688

Closed
stu177 opened this issue Feb 1, 2016 · 15 comments · Fixed by #693
Closed

New UI hover effects #688

stu177 opened this issue Feb 1, 2016 · 15 comments · Fixed by #693
Assignees
Milestone

Comments

@stu177
Copy link

stu177 commented Feb 1, 2016

The new hover effects in 0.10.0 to me seem to hinder usability. I found the buttons before much easier to use and navigate. Maybe it's just me but wonder if other people feel the same.

@barbarah
Copy link

barbarah commented Feb 1, 2016

I'm also not a great fan of the new UI hover effects. For me the biggest problem is that I cannot see to which fork I'm pushing to. The fork you push to is the fork selected in the top right, when I hover over the push button the fork name in the top right is hidden. Furthermore the new effects feel a bit messy/chaotic. I don't think it's an improvement.

@campersau
Copy link
Collaborator

For reference: This behaviour got introduced with #550

@jung-kim jung-kim added this to the 0.10.1 milestone Feb 1, 2016
@jung-kim jung-kim self-assigned this Feb 1, 2016
@jung-kim
Copy link
Collaborator

jung-kim commented Feb 1, 2016

Hmm... I do apologize, and intention was to reduce the overall width of the repository-buttons group's width. I do agree that having target repo displayed all the time is helpful as push op target is depends on that.

But do you feel that it is messy/chaotic due to timing of the expansion and reduction or do you not like the this iconized effort at all?

@chrisuehlinger
Copy link

I'm a big fan of this tool and recommend it to everyone at work (including designers and other people who are not hardcore developers). I recommend it based on its user-friendliness, but also because it takes the words people have heard ("I'll rebase that tomorrow", "We need more review before we merge", "I pushed that on Tuesday") and puts them in a graphical context they can understand. With the words hidden behind hover effects, the value of ungit as a teaching tool is noticeably hindered, and there isn't another tool I can recommend to people instead.

I also love ungit because of its attention to the mobile experience. Hover effects won't work on my iPad, and so if I pull up a repo on my iPad to show someone, they need to already know what the icons mean in order to understand what I'm showing them.

If you guys really feel like there's a need for the hover effects, can you at least add a config option to turn them off (or even make them an opt-in for power users)?

@daedalus28
Copy link
Contributor

I am also a huge fan of the tool, and with this update for patch support I think it's literally the best git client, period. There's nothing left that I miss from SourceTree or any other client I've used before (except maybe first class support for git-flow).

With that said, I'm afraid I need to second the sentiment regarding the hover effects. In general, "mystery meat" icon navigation only works when the icons are both immediately obvious as to what they represent and also ideally have some precedent in the user's mind for what they ought to do. In the case of ungit, much of the value lies in its accessibility when users may not fully understand what's going on (for example, I've never fully understood rebase until I saw it so clearly laid out in ungit).

I may be a little biased though because on my giant 4K screen there's no need to try to save any space at all :)

@stu177
Copy link
Author

stu177 commented Feb 1, 2016

I'd prefer if they were just buttons with no hover effect, its the way when you hover over something it pushes everything along horizontally so the buttons are always moving all over the place.

@FredrikNoren
Copy link
Owner

Thanks for the feedback everyone! @codingtwinky +1 to fixing this for 0.10.1, thank's for labeling.

Also going forward I think we should try to avoid hover effects in general. We also have hover effects on the branch, submodule and remote drops downs, which I think we could probably solve in a more elegant way without hover effects.

@jung-kim
Copy link
Collaborator

jung-kim commented Feb 2, 2016

I absolutely agree that one of the ungit's most attractive feature is ease of use and help users undestand git. Hover action does hide details that matters and possibly obscure and confuse.

I will remove hover feature and display full details for all of the buttons.

One quick Q though. How about setting maximum width to buttons and have it expand on hover for few select buttons? It is a potential problem that having long branch name can invasive horizontally. Or is this a miniscule edge case?

@stu177
Copy link
Author

stu177 commented Feb 2, 2016

I think steering clear of hover effects is the best route here. You could truncate extremely long branch names to deal with this issue, but I do think it is quite an edge case and isn't much to worry about.

@chrisuehlinger
Copy link

Bear in mind that horizontal space is most scarce on mobile/tablet devices, which also (because they are touch-based) can't handle hover effects.

@stu177
Copy link
Author

stu177 commented Feb 3, 2016

I don't think mobile/tablet should be considered as much as desktop in my opinion. I would imagine the majority of ungit users are on a desktop/laptop? I could be wrong though.

@jung-kim
Copy link
Collaborator

jung-kim commented Feb 3, 2016

For "main" view I do think that it would be best to focus on desktop display.

However, I do not want to abandon mobile users and we should detect mobile device automatically, which is little crazy, and do serve more mobile specialized view.

@chrisuehlinger is there any thing you wish to be added or removed for mobile view? I can't think of any for me now but I don't use ungit on mobile devices. I can work with you to deliver more customized view for mobiles.

if you do please open new issues.

@FredrikNoren
Copy link
Owner

99.88% of all Ungit users (that have enable the google analytics usage tracking) are on desktop, so I think we don't need to put too much effort on mobile just yet :)

@chrisuehlinger
Copy link

Oh, my point had to do with the fact that ungit works beautifully on mobile as is (before hover effects). I'm certain that I'm in the minority, but I often use ungit from my iPad to conserve screen space on my desktop, and it's awesome.

@FredrikNoren
Copy link
Owner

@chrisuehlinger totally. I also think that designing for touch screens forces things to be simpler too, so think it's good practice to just do it even if the usage doesn't warrant it directly

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 a pull request may close this issue.

7 participants