derekstavis / transmission

A GNOME 3.14 HIG-compliant Transmission
Other
55 stars 1 forks source link

The app menu can be simplified. #2

Closed me4oslav closed 9 years ago

me4oslav commented 9 years ago

So, I compiled this branch yesterday and YAY, headerbar and symbolic icons for transmission: https://github.com/derekstavis/transmission/tree/wip/gtk-3.12 However when I opened the app menu from the top left on the gnome-shell panel I got ... well, see the attached file. As you can imagine this menu can be bug and confusing for some people and I would imagine you don't want this to happen. screenshot from 2015-02-12 12 05 41 First three option are reduntant - New torrrent si already in the headerbar and you can make the "open" button in it toggle a popover with: open file open url The second three can stay, although I personally see no point of the compact view mode and disabling the statusbar. Monitor nowadays are big enough. However I am not sure how will you would be to nuke these two options, because I know from experience that open source folks do enjoy having extra bells and whilstles here and there. The third three. The Edit -> de/select all should go in the regular hamburger menu to follow the GNOME HIG. Nautilus does that and I guess you want to stick to GNOME's HIG. Start all and Pause all can stay. However the start/pause button in the headerbar should be simply for a single torrent and not a start all/pause all button. The rest of the items are just fine.

So, what do you think of my suggestions? Cheers!

aldomann commented 9 years ago

Relevant links: Application menus guidelines: https://developer.gnome.org/hig/stable/application-menus.html.en Header bar (aka hamburger) menus guidelines: https://developer.gnome.org/hig/stable/header-bar-menus.html.en

derekstavis commented 9 years ago

Thanks for compiling and testing it, that's awesome! I'm planning a menu redesign and your point of view is very important.

I'm still unsure about Message Log, Statistics and Donate.

me4oslav commented 9 years ago

@derekstavis Thanks for the quick response! (feels odd for me to say this, when I am usually the guy that is at the receiving end of that sentence ;p) Ace to see the huge percentage of the suggestions will be implemented, looking forward to testing the version with the redesigned menu, so do ping me here or on Google+ when they've landed. :) Maybe next up we can redo the preferences and I've some categorisation ideas for the headerbar too. But let's take it one step at a time and first see it with cleaned up redesigned app menu. P.S. I don't have a lot of free time to spare, but by Jove - transmission needs UI polishing and I will try to help you as much as possible with your crusade.

derekstavis commented 9 years ago

@me4oslav great! there's lots of improvements to be made! thanks again! As soon as I can get some free time to implement this changes (probably today) I ping you!

me4oslav commented 9 years ago

http://www.reactiongifs.com/wp-content/uploads/2014/01/fantastic.gif

me4oslav commented 9 years ago

Oh and shouldn't "Contents" become Help?

derekstavis commented 9 years ago

Yeah! So, the proposal looks like this:


Compact View Status Bar Preferences


Message Log Statistics Donate


Help About Quit

me4oslav commented 9 years ago

I guess this would be better:

Pause all*


Compact view Status bar Preferences


Message lg Statistics Donate


Hep About Quit

*That should become "Start all" if all torrents are paused.

And don't forget to add the necessary headerbar fixes: 1) The open file button should open a popover with two options: "open torrent" and "open url" 2) The start/pause button on the headerbar should start/pause a single torrent, because right now there isn't a way to do that. If one wants to pause all - the top option at the appmenu is for him/her.

derekstavis commented 9 years ago

Makes total sense, altough I don't know how simple Start All/Pause All is to implement for a first release. Anyway, the behavior of Start/Pause button in headerbar is identical to old one and OS X Transmission client, so I don't know how acceptable it would be for users after updating.

aldomann commented 9 years ago

Wouldn't it be easier to implement tick boxes or any other way to have multiple selections? That would integrate much better with the GNOME HIG: https://developer.gnome.org/hig/stable/selection-mode.html.en

derekstavis commented 9 years ago

Selection mode would be awesome

derekstavis commented 9 years ago

About app menu, it seems great. Compiling, for a first implementation:


Compact View Status Bar


Preferences


Message Log Statistics


Donate Help About Quit

me4oslav commented 9 years ago

Well, worst case scenario - keep Pause all and Start all for now. @derekstavis Sure this is the current transmission behaviour? To not have a button for pausing or resuming a single torrent? :o I am like 101% sure it has one. @aldomann Selections for what?

derekstavis commented 9 years ago

@me4oslav It has in form of a right-click popup menu. For the toolbar it's worst: It haves a button for starting and a button for pausing.

me4oslav commented 9 years ago

@derekstavis Yes, but these two buttons on the toolbar control the start/pause for a single torrent and not for all. Hence the start/pause button in the headerbar in here should be for single torrent too.

aldomann commented 9 years ago

@me4oslav for selecting multiple torrents at once and the action bar would contain: Play, Pause, Delete, Delete and Remove, Properties.

Let me do some quick mock-ups to illustrate better my idea.

me4oslav commented 9 years ago

@aldomann Ah, I got your idea.

me4oslav commented 9 years ago

@derekstavis @aldomann Ta: img_20150212_163251 img_20150212_163259 Apologies about the handwriting. :)

aldomann commented 9 years ago

That's cheating! I was (and still am, alas) making digital wireframes.

me4oslav commented 9 years ago

Which is why I use the forgotten art of using pen and paper. :P Digital stuff takes more time and in 99% of the cases hand drawn wireframes and good enough for devs. Anyways handwriting aside, what do you guys make of these?

aldomann commented 9 years ago

Done! Actually, when you think about it, it makes sense to have an always active selection mode; the use of a permanent action bar is relevant in Transmisison, me thinks. transmission-wire

The view-list button should show a popup to sort the downloads and stuff related of the view mode.

PS. The numbers and lenght of the progress bars actually add up :P

me4oslav commented 9 years ago

@aldomann So, here I go: I like the always active selection mode, but why are the start, pause, properties, remove and delete and remove in a separate bar at the bottom? Can we stack the the same way I've stacked them in my headerbar wireframe. No need to waste screen space with one more bar, really. Best of both words. What does the turtle button do? :o The rest is ok, I guess.

aldomann commented 9 years ago

Man, the magic turtle!!! It limits your download/upload speeds set in your settings. You are right about the action bar. Actually the HIG recommends not to use it if they are permanent actions. "Header bars are dynamic" so, showing the right buttons when necessary shouldn't be a problem; if there's no selected torrent, no "start/pause, delete/delete & remove, properties" buttons need to be shown.

In the worst case scenario, those could be insensitive when not needed (no idea if the HIG recommends not to do this, though).

me4oslav commented 9 years ago

@aldomann, I've never seen a turtle button before. :P Now I know what it does. I guess we're ok with start/pause, delete/delete and remove and properties being inactive when there is no torrent selected. They won't be of any use to be active in this case. Just one minor thing - clicking anywhere in the space the torrent take should selected it, so users will not have to hit that pesky checkbox for a single torrent selection. Now, create a new wireframe with all the changes. :)

derekstavis commented 9 years ago

Lots of great ideas! That's pretty awesome! But hey, we need to keep the discussion organizated. Let's keep this issue with comments about the Application Menu. You may create other issues for proposing alternative layouts for main window.

me4oslav commented 9 years ago

@derekstavis it will look even more awesome in the newest wireframe that I shamelessly dispatched Alfredo to do :) And yes, I'll create another issue for the main window and headerbar designs. Just need the final wireframe to do so.

derekstavis commented 9 years ago

Great. I see we can make Transmission super-awesome. I just need to deal with technical issues in the code, and find free time to work on it. At the same time, I need to consume scope, as the first release will be the base of my term paper, that's already in progress, and I can't add more features right now :)

me4oslav commented 9 years ago

I don't think adding features are some sort of major concern. Transmission is in needs of a visual facelift much more than in need of new features. So don't worry about new features just yet. Let's finish the visuals first and then try to get the design upstream. Then we can start thinking for extra features.

derekstavis commented 9 years ago

Sorry, by extra features I meant design features :)

derekstavis commented 9 years ago

I need to finish my term paper, and I already defined a scope. Some fixes can be accepted, like the Application Menu and Start/Pause behavior, but I can't add major things, like selection mode, even if I think it's awesome. I hope you guys understand this and have some patience before I can start working on more visual facelifts :)

me4oslav commented 9 years ago

Ah. :P Well, we can take it one step at a time, it is not like any of us is getting paid for this, so we can pull of a Danny Boy (Daniel from elementary, why I call him Danny Boy is a long story) and adopt the "when it's ready" agenda. Plus knowing myself and Alfredo we can popularise the designs when we finish them and maybe attract devs to help you.

derekstavis commented 9 years ago

I just started on OSS, my mission is to GTKalize and CSDlize everything, but I don't have any developer network to help me with this :)

me4oslav commented 9 years ago

We can try to lure dev hamsters, but no promises given that it will actually work. But it is worth a shot, because open source can surely use less sucky design.

aldomann commented 9 years ago

@derekstavis that's also my mission (although I'm no developer)! Check my deviantArt page: http://aldomann.deviantart.com/gallery/

We (@me4oslav and me) can help you with designs any time you need :)

derekstavis commented 9 years ago

I like to design and like to develop. We will probably get best friends :+1:

me4oslav commented 9 years ago

And my mission is to keep Starling City safe. :D I very mmuch enjoy sleeping. And drinking. And eating. And reading. In no specific order. @aldomann Waiting for that new wireframe for the main layout and headerbar.

derekstavis commented 9 years ago

I sleep a lot too. That's why Transmission isn't yet ready.

aldomann commented 9 years ago

@me4oslav http://aldomann.deviantart.com/art/Transmission-GTK-3-CSD-redesign-513447880

derekstavis commented 9 years ago

Also, this code is old. It doesn't even use builder XML.

me4oslav commented 9 years ago

@derekstavis Sleeping is absolutely brilliant. :) @aldomann Just add the appmenu there.

me4oslav commented 9 years ago

Ta - https://github.com/derekstavis/transmission/issues/3

derekstavis commented 9 years ago

App menu was simplified. You can check it on branch wip/gtk3.14.