Loomio
Sat 25 Feb 2017 9:32PM

Cleaning up Diaspora's visual design

ST Sean Tilley Public Seen by 138

I have had some interest recently in polishing up Diaspora's default look and feel. I believe that some minor changes to stylesheets and templates could greatly improve the default user experience of using Diaspora.

My line of inquiry has come to this: at this point, how does one go about proposing visual changes to Diaspora's interface? I feel competent enough in working with the many compartmentalized feature-specific stylesheets, and have some understanding of interface elements in various templates.

Put another way: if I have ideas and can make most of them happen on a branch, what is the ideal review process for submitting a series of changes? Is it better to focus on one set of changes per pull request (ie, "switch box shadows for 1px solid borders?") or is a PR replete with screenshots and multiple commits enough?

I've had some interest in taking a "100 Papercuts" approach here, where minor design squabbles across the UI are listed to be worked on. I'm not 100% sold on the approach, however.

Any thoughts or suggestions on how to go about this?

ST

Sean Tilley Sat 25 Feb 2017 9:38PM

I have a branch that I'm working on here, only been at it for a few days so far: https://github.com/DeadSuperHero/diaspora/tree/theme-cleanup

CK

Corey Kimball Sat 25 Feb 2017 9:51PM

Hey Sean- my suggestions are such: making revisions to the user-interface, and pushing for approval should be focused on modular design and areas of the platform, that way.

Regards, we should talk soon,

-Corey

DS

Dennis Schubert Sat 25 Feb 2017 11:04PM

If you can, make a mockup/screenshot of your idea first and propose them in a newly created github issue or here. If you cannot, make PRs (and include screenshots if you want <3), but keep them as isolated as possible.

The smaller the PR, the easier reviews are and the lower is the risk for massive refactorings if we decide to change something.

S

SuperTux88 Sat 25 Feb 2017 11:37PM

The smaller the PR, the easier reviews

Multiple small commits (with one change per commit and meaningful commit-message) are also easy to review :)

Also @comradesenya made a public test-pod (see https://www.loomio.org/d/xzs7W59r/diaspora-crowdtesting), maybe it is possible to deploy your PR there and get feedback from the community (But senya is currently testing his jquery PR there).

JS

Juan Santiago Sun 26 Feb 2017 2:24AM

Screenshot please.

ST

Sean Tilley Fri 3 Mar 2017 8:38AM

@juansantiago Feast your eyes and your feedback: https://github.com/diaspora/diaspora/pull/7349