Skip to content
This repository has been archived by the owner on Apr 17, 2024. It is now read-only.

Change Contenta theme colors #175

Closed
yann-yinn opened this issue Aug 26, 2017 · 19 comments
Closed

Change Contenta theme colors #175

yann-yinn opened this issue Aug 26, 2017 · 19 comments

Comments

@yann-yinn
Copy link
Contributor

yann-yinn commented Aug 26, 2017

Hello !
Default colors from the theme are pretty dark and a little bit sad. A graphist friend of mine ( @FlorisMoriceau ), suggested this design change for the colors : he uses the logo color for the header and a lighter background, I find this color palette more engaging, what do you think about it ?

Actual :
contenta-colors

Proposal :
contenta-new-colors

@e0ipso e0ipso changed the title Lightening Contenta theme colors Change Contenta theme colors Aug 26, 2017
@e0ipso
Copy link
Member

e0ipso commented Aug 26, 2017

Paging @ckrina for an opinion on this.

@ckrina
Copy link

ckrina commented Aug 26, 2017

Thanks @e0ipso :)
I totally agree with @nyl-auster and @FlorisMoriceau about the the "sad" perception of the interface: I think everything is a bit dark, and also the greys are too green. On this screenshot the page looks much more cleaner.
But I'm not sure if I'd use the bright blue on the header. This color was picked to be the primary button color in forms so it could stand out among the other elements. I agree that it's much more appealing, but seeing it all the time can be tiring. Maybe just using it in some places could help on that?

@e0ipso
Copy link
Member

e0ipso commented Aug 26, 2017

FYI this is the current look of Contenta: http://cms.contentacms.io and http://cms-dev.contentacms.io

@yann-yinn
Copy link
Contributor Author

yann-yinn commented Aug 26, 2017

@e0ipso it's nice ! this is not the one i saw when installing contenta following this process ,do you know why ? https://yineo.fr/blog/en-how-to-install-contenta-cms-on-pantheon-with-terminus

@e0ipso
Copy link
Member

e0ipso commented Aug 27, 2017

@nyl-auster I believe that Pantheon is fixed to an old commit.

Maybe @greg-1-anderson can clarify.

@yann-yinn
Copy link
Contributor Author

@e0ipso you're right, from a fresh install i got this , so i am closing this issue
capture d ecran 2017-08-27 a 13 59 06

@greg-1-anderson
Copy link
Contributor

greg-1-anderson commented Aug 28, 2017

I couldn't read the blog above - got a server error when I clicked on the link.

The Pantheon experimental Contenta installation project is pinned to dev-8.x-1.x. I'd be happy to change that to something else if there's a more appropriate version constraint now.

@yann-yinn
Copy link
Contributor Author

@greg-1-anderson yes i unpublished my blog post because i would not recommend this version of contenta ( as i opened 3 issues for things that have been resolved in current contenta version)

@greg-1-anderson
Copy link
Contributor

What is the "current contenta version", though? It looks to me as is 8.x-1.x is still the appropriate branch for dev releases? Maybe you installed the dev version a while ago? If so, you might simply need to update your site. Use composer update on a local working copy of your site's GitHub repository.

It would be helpful if there were SemVer release tags for the contenta distribution.

@yann-yinn
Copy link
Contributor Author

yann-yinn commented Aug 28, 2017

@greg-1-anderson

I dont know about the versions ( cc @e0ipso ), just for information :

this is the command I ran this week-end with terminus

terminus build:project:create pantheon-systems/example-drops-8-composer:dev-contenta YOUR-SITE-NAME

And then i ran this command in my localhost to reinstall it from scratch

composer create-project contentacms/contenta-jsonapi-project MYPROJECT --stability dev --no-interaction

@greg-1-anderson
Copy link
Contributor

Not sure if this is what you were trying to say, but that second command is not an appropriate way to update a Pantheon Contenta site. You could use that composer create-project to create a non-Pantheon Contenta site for local use. To update a site created with terminus build:project:create, use composer update.

@yann-yinn
Copy link
Contributor Author

yann-yinn commented Aug 28, 2017

@greg-1-anderson

I used the first command to create automatically a pantheon site. And that works very well but it gave me the first screenshot from this issue, and i was missing some modules ( snail ). I dit not try to update the pantheon site at this point, because, well, it was freshly installed. I told myself that maybe terminus must have built something from an outdated contenta release tag.

From then, i deleted my pantheon site and I used the second command on my local and saw that it gave me a version of contenta with the right theme and modules I needed.

Shit this issue is a mess, i am so sorry people, just wanted to give a bit of light to the dark theme xD

@greg-1-anderson
Copy link
Contributor

I would think those two commands should give you about the same Contenta, given that you ran them at the same time. Maybe add --stability dev to the Terminus command, as you have in the composer create-project command? That could explain some differences in project versions.

@e0ipso
Copy link
Member

e0ipso commented Aug 30, 2017

@greg-1-anderson can this be related to composer.lock using an old version?

pantheon-systems/example-drops-8-composer@c6ab421#diff-1da2c7edc898c70e5a79a9997c98ceccR392

@greg-1-anderson
Copy link
Contributor

Yeah, makes sense. composer update will fix that.

@e0ipso
Copy link
Member

e0ipso commented Aug 30, 2017

@nyl-auster can you try that and see if it works? Maybe you can re-post the article after that.

@yann-yinn
Copy link
Contributor Author

@e0ipso sorry i already deleted this site. But i guess i can just add "composer update" to my blog post ?

@greg-1-anderson
Copy link
Contributor

I'll remove the composer.lock file from the PR

@greg-1-anderson
Copy link
Contributor

Done; tests are running at pantheon-systems/example-drops-8-composer#93

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

No branches or pull requests

4 participants