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

Finish Contributions Manager UI design #3482

Closed
alignedleft opened this Issue Jul 18, 2015 · 23 comments

Comments

Projects
None yet
7 participants
@alignedleft
Member

alignedleft commented Jul 18, 2015

For #3464 and other issues, we need to finalize the visual design of the new Contributions Manager.

I've started on mockups, but before I go further, I would like your input on whether or not this general approach matches the new editor stylistically. I've reached out to James by email, but in the meantime have lifted color values and approximated other design elements.

Would appreciate comments from: @benfry @REAS @shiffman

artboard 2

Thanks!

@alignedleft alignedleft self-assigned this Jul 18, 2015

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Jul 19, 2015

Member

I think it would be worth looking at this screenshot, most of the libraries have long description(even if we take only the first line) so the CM won't look as clean(with the default font size).
screenshot from 2015-07-19 14 15 56

Member

Akarshit commented Jul 19, 2015

I think it would be worth looking at this screenshot, most of the libraries have long description(even if we take only the first line) so the CM won't look as clean(with the default font size).
screenshot from 2015-07-19 14 15 56

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Jul 20, 2015

Member

@Akarshit Yes, thanks. My dummy text above doesn't reflect the real data. I'd like for text that doesn't fit to be abbreviated with an ellipsis ("…"), as you have done above. Then selecting that line will reveal the full description in the lower panel (as you have it working now).

Member

alignedleft commented Jul 20, 2015

@Akarshit Yes, thanks. My dummy text above doesn't reflect the real data. I'd like for text that doesn't fit to be abbreviated with an ellipsis ("…"), as you have done above. Then selecting that line will reveal the full description in the lower panel (as you have it working now).

@shiffman

This comment has been minimized.

Show comment
Hide comment
@shiffman

shiffman Jul 23, 2015

Member

Personally I love how this looks! I think I understand the icons (installed and updated, installed but needs an update, not compatible) but what do the greyed out ones indicate? Not compatible and not installed?

Member

shiffman commented Jul 23, 2015

Personally I love how this looks! I think I understand the icons (installed and updated, installed but needs an update, not compatible) but what do the greyed out ones indicate? Not compatible and not installed?

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Jul 24, 2015

Member

@shiffman the greyed out once are those which are not compatible. The triangular icon is for installed and not compatible.

Member

Akarshit commented Jul 24, 2015

@shiffman the greyed out once are those which are not compatible. The triangular icon is for installed and not compatible.

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Jul 24, 2015

Member

@shiffman Yes, sorry for not explaining. This mockup mirrors the wireframes I had emailed around earlier (with more clear explanations). @Akarshit’s explanation is correct. Thanks for the feedback!

Member

alignedleft commented Jul 24, 2015

@shiffman Yes, sorry for not explaining. This mockup mirrors the wireframes I had emailed around earlier (with more clear explanations). @Akarshit’s explanation is correct. Thanks for the feedback!

@clankill3r

This comment has been minimized.

Show comment
Hide comment
@clankill3r

clankill3r Jul 29, 2015

Can't install / update not be merged in to one button and show whatever is appropriate for the situation?

clankill3r commented Jul 29, 2015

Can't install / update not be merged in to one button and show whatever is appropriate for the situation?

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Jul 29, 2015

Member

@clankill3r Great point. I think you're right, although there was some reason why we went with three buttons here instead of three (not ideal UI, I know, but I remember something about it being simpler to have three separate buttons than fewer buttons who change their behavior dynamically). If anyone can think of a reason not to merge these buttons, please let us know here.

Member

alignedleft commented Jul 29, 2015

@clankill3r Great point. I think you're right, although there was some reason why we went with three buttons here instead of three (not ideal UI, I know, but I remember something about it being simpler to have three separate buttons than fewer buttons who change their behavior dynamically). If anyone can think of a reason not to merge these buttons, please let us know here.

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Aug 3, 2015

Member

@REAS @benfry any comments? If not, I'm going to move forward with this, as we're nearing the end of GSoC, and @Akarshit needs to know what to work on next. Thanks.

Member

alignedleft commented Aug 3, 2015

@REAS @benfry any comments? If not, I'm going to move forward with this, as we're nearing the end of GSoC, and @Akarshit needs to know what to work on next. Thanks.

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Aug 3, 2015

Member

I think it's close enough that you guys should charge ahead with it, and James will be providing some feedback, but it'll be less structural (so it should have less impact on @Akarshit's work, or be quicker modifications once this general setup is in place).

Member

benfry commented Aug 3, 2015

I think it's close enough that you guys should charge ahead with it, and James will be providing some feedback, but it'll be less structural (so it should have less impact on @Akarshit's work, or be quicker modifications once this general setup is in place).

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Aug 3, 2015

Member

Super, thanks, @benfry. I'm communicating with @Akarshit separately to get this implemented.

Member

alignedleft commented Aug 3, 2015

Super, thanks, @benfry. I'm communicating with @Akarshit separately to get this implemented.

@jamesjgrady

This comment has been minimized.

Show comment
Hide comment
@jamesjgrady

jamesjgrady Aug 4, 2015

Contributor

@alignedleft @benfry here is the latest Contributions Manager UI comp. Let me know if you have comments/revisions. Also, let me know if you will need individual files for the Processing Foundation icons or the other icons (i.e. Installed, Update, Error)

pde-contmgr-001

Contributor

jamesjgrady commented Aug 4, 2015

@alignedleft @benfry here is the latest Contributions Manager UI comp. Let me know if you have comments/revisions. Also, let me know if you will need individual files for the Processing Foundation icons or the other icons (i.e. Installed, Update, Error)

pde-contmgr-001

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Aug 4, 2015

Member

Thank you @jamesjgrady! This looks really nice. However, there are several different states that this also needs to account for. I'll send you an email now with the full set of wireframes, which illustrate all the needed functionality. This might need minor adjustments to accommodate.

Member

alignedleft commented Aug 4, 2015

Thank you @jamesjgrady! This looks really nice. However, there are several different states that this also needs to account for. I'll send you an email now with the full set of wireframes, which illustrate all the needed functionality. This might need minor adjustments to accommodate.

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Aug 5, 2015

Member

Which design is to be implemented the one originally given by @alingedleft or the one given by @jamesjgrady

Member

Akarshit commented Aug 5, 2015

Which design is to be implemented the one originally given by @alingedleft or the one given by @jamesjgrady

@jamesjgrady

This comment has been minimized.

Show comment
Hide comment
@jamesjgrady

jamesjgrady Aug 5, 2015

Contributor

I have to look at the other wireframes and see where adjustments need to be made, but this latest comp is the general direction for design. I hope to have revised comps later today.

Contributor

jamesjgrady commented Aug 5, 2015

I have to look at the other wireframes and see where adjustments need to be made, but this latest comp is the general direction for design. I hope to have revised comps later today.

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Aug 8, 2015

Member

@alignedleft @jamesjgrady
Is the display "Version 0.9 available" below the install button removed ?
What about the progress bar, is it removed too ?

Member

Akarshit commented Aug 8, 2015

@alignedleft @jamesjgrady
Is the display "Version 0.9 available" below the install button removed ?
What about the progress bar, is it removed too ?

@JukePlz

This comment has been minimized.

Show comment
Hide comment
@JukePlz

JukePlz Aug 8, 2015

Is there really a point for separate install and remove buttons? Shouldn't they be different states for the same button?

JukePlz commented Aug 8, 2015

Is there really a point for separate install and remove buttons? Shouldn't they be different states for the same button?

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Aug 8, 2015

Member

@alignedleft the CM looks like
this
. If you could send me the finalized designs and icons it would help me to finalize the CM.
What is the design of the filter and category strip ? ( it has been changed 3 times and i am still not sure which one is correct)

Member

Akarshit commented Aug 8, 2015

@alignedleft the CM looks like
this
. If you could send me the finalized designs and icons it would help me to finalize the CM.
What is the design of the filter and category strip ? ( it has been changed 3 times and i am still not sure which one is correct)

@clankill3r

This comment has been minimized.

Show comment
Hide comment
@clankill3r

clankill3r Aug 8, 2015

Is there really a point for separate install and remove buttons? Shouldn't they be different states for the same button?

Good point! That would be nice.

Also, the blue color of the highlighted line is a way to dark blue color. It is really hard to read (on my monitor).

For the rest it looks nice.

I was wondering, will there be a new way to go to this menu?
Now you either have to use "add tool..." or "add mode..." to get there.

clankill3r commented Aug 8, 2015

Is there really a point for separate install and remove buttons? Shouldn't they be different states for the same button?

Good point! That would be nice.

Also, the blue color of the highlighted line is a way to dark blue color. It is really hard to read (on my monitor).

For the rest it looks nice.

I was wondering, will there be a new way to go to this menu?
Now you either have to use "add tool..." or "add mode..." to get there.

@Akarshit

This comment has been minimized.

Show comment
Hide comment
@Akarshit

Akarshit Aug 8, 2015

Member

@clankill3r the blue color will be surely changed in next release.

Member

Akarshit commented Aug 8, 2015

@clankill3r the blue color will be surely changed in next release.

@clankill3r

This comment has been minimized.

Show comment
Hide comment
@clankill3r

clankill3r Aug 8, 2015

This is also really hard to read:

screen shot 2015-08-08 at 2 36 59 pm

clankill3r commented Aug 8, 2015

This is also really hard to read:

screen shot 2015-08-08 at 2 36 59 pm

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Aug 8, 2015

Member

Locking to reduce noise until we can finish implementation.

Member

benfry commented Aug 8, 2015

Locking to reduce noise until we can finish implementation.

@processing processing locked and limited conversation to collaborators Aug 8, 2015

@benfry benfry changed the title from Contributions Manager UI design to Finish Contributions Manager UI design Aug 9, 2015

@alignedleft

This comment has been minimized.

Show comment
Hide comment
@alignedleft

alignedleft Aug 13, 2015

Member

@Akarshit Re: the filter and category strip, please reference the most recent mockup from James above. Use that alignment, widths, and colors/grays. We can get the final icon images from @jamesjgrady and add them in later.

Member

alignedleft commented Aug 13, 2015

@Akarshit Re: the filter and category strip, please reference the most recent mockup from James above. Use that alignment, widths, and colors/grays. We can get the final icon images from @jamesjgrady and add them in later.

@benfry benfry added this to the 3.0 final milestone Aug 14, 2015

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Sep 21, 2015

Member

Closing this since I think we're mostly there, except for other more specific issues that have been filed.

If anything is missing, please file individual issues for each specific problem so we can sort it out.

Member

benfry commented Sep 21, 2015

Closing this since I think we're mostly there, except for other more specific issues that have been filed.

If anything is missing, please file individual issues for each specific problem so we can sort it out.

@benfry benfry closed this Sep 21, 2015

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.