Loomio
February 25th, 2017 21:32

Cleaning up Diaspora's visual design

Sean Tilley
Sean Tilley Public Seen by 490

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?

Sean Tilley

Sean Tilley February 25th, 2017 21:38

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

Corey Kimball

Corey Kimball February 25th, 2017 21:51

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

Dennis Schubert

Dennis Schubert February 25th, 2017 23:04

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.

SuperTux88

SuperTux88 February 25th, 2017 23:37

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).

Juan Santiago

Juan Santiago February 26th, 2017 02:24

Screenshot please.

Sean Tilley

Sean Tilley March 3rd, 2017 08:38

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