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

Better style #116

Closed
wants to merge 1 commit into from
Closed

Better style #116

wants to merge 1 commit into from

Conversation

eVanilla
Copy link

@eVanilla eVanilla commented Feb 7, 2018

Is everything all right asking?
Well. Almost 1 month ago I made a request and nobody cared for it, and I decided to do it myself.
And I know it's a little work, but I wanted to show you guys that I also have other ideas for improving ui & stuff.

@eVanilla eVanilla mentioned this pull request Feb 9, 2018
@Flaburgan
Copy link
Member

Sorry I wanted to check your PR but my dev env is currently broken. I'll try to have a look in the coming days.

@Flaburgan
Copy link
Member

Flaburgan commented Mar 31, 2018

So, I finally succeed to run the official website locally, to try your PR.
It looks like that:
screenshot-2018-3-31 le projet diaspora

So it looks like you have changed:

  • The header background, going from black to white
  • The "hero" background, replacing the grey by a gradient
  • The background of the page, from white to grey, adding some box around the text, with white background and shadow.
  • The footer background from white to black

That's what I spotted with a very quick look. Did I miss something?

My first feedback would be partially negative unfortunately. I don't think we should do the 2 first changes I listed mainly for the following reason: we're trying to keep a common identity between the official website and diaspora* itself. The colors are well defined: the header is black, the buttons are blue or green, etc. From that point of view, a white header followed by a purple hero isn't what we want in my opinion.

However, your two last changes are interesting: the background which was white in diaspora* has been changed to grey with the content put in white box since the version 0.6.0.0, exactly like you did here. The website hasn't been updated with that change, so maybe we want to do it. I'll wait for others' opinion on that point before giving you a go, but that change can be a good one.

You also changed the footer from white to black. In diaspora*, it is grey. I don't know what we want to do here.

In any case, thank you for helping improve diaspora* and sorry again for the very late answer.

@Flaburgan
Copy link
Member

I actually tried the grey background for the footer in #119. It feels better that way imo, more unified with diaspora*.

@SuperTux88
Copy link
Member

From that point of view, a white header followed by a purple hero isn't what we want in my opinion.

I agree with that, also I think it shouldn't be that big (it covers almost half of the page) ... on every page, so you need to scroll on some pages now, where you didn't need to scroll before:

image

before:

image

@Flaburgan
Copy link
Member

@eVanilla #119 has been merged so the footer is now unified, as you proposed. I can still be interested by the changes from white to grey background with white shadowed boxes for content, I'm not sure (maybe at least not in the home page, I plan to change its content).

@denschub denschub closed this Jun 13, 2020
@denschub
Copy link
Member

This PR got automatically closed by GitHub after we deleted the master branch and replaced the default branch with production. I did miss that this change will close all open PRs, sorry for that. Since there has been no activity for a while, I think this is fine, but if someone feels strongly about this, please let me know and we can restore this. :)

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 this pull request may close these issues.

5 participants