Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 440663 - PlanetEclipse Needs to update to Solstice
Summary: PlanetEclipse Needs to update to Solstice
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PlanetEclipse (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Christopher Guindon CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 440814 442514
Blocks:
  Show dependency tree
 
Reported: 2014-07-29 11:42 EDT by Ian Skerrett CLA
Modified: 2019-08-20 10:43 EDT (History)
3 users (show)

See Also:


Attachments
Screenshot (249.49 KB, image/png)
2014-08-01 14:21 EDT, Christopher Guindon CLA
no flags Details
Example logos for PlanetEclipse (262.41 KB, application/pdf)
2014-08-05 12:28 EDT, Ian Skerrett CLA
no flags Details
Screenshot with new logo (356.57 KB, image/png)
2014-08-05 12:40 EDT, Christopher Guindon CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Skerrett CLA 2014-07-29 11:42:14 EDT
We need to update the banner of PlanetEclipse to be more inline with the new logo and Solstice.
Comment 1 Denis Roy CLA 2014-07-30 10:35:11 EDT
I originally thought that Planet could stay "as-is" since it's more of a community site, but I suppose it would look good with the Solstice theme.
Comment 2 Ian Skerrett CLA 2014-07-30 10:38:42 EDT
(In reply to Denis Roy from comment #1)
> I originally thought that Planet could stay "as-is" since it's more of a
> community site, but I suppose it would look good with the Solstice theme.

At a minimum we need to update the logo. Should I ask Matt to design a new header?
Comment 3 Denis Roy CLA 2014-07-30 10:40:17 EDT
> At a minimum we need to update the logo.

You're right, I had missed that part.  I suggest we use Solstice.

In the future I will just +1 whatever you say.
Comment 4 Ian Skerrett CLA 2014-07-30 10:41:19 EDT
(In reply to Denis Roy from comment #3)
> > At a minimum we need to update the logo.
> 
> You're right, I had missed that part.  I suggest we use Solstice.
> 
> In the future I will just +1 whatever you say.

Finally, respect.
Comment 5 Christopher Guindon CLA 2014-08-01 14:21:18 EDT
Created attachment 245657 [details]
Screenshot

I have a patch ready but before we go live with this, we need a new planeteclipse logo.

https://git.eclipse.org/r/#/c/30900/
Comment 6 Denis Roy CLA 2014-08-01 14:37:19 EDT
Looks good!  Just a nit, I think the breadcrumbs should be Home > Community > Planet Eclipse
Comment 7 Christopher Guindon CLA 2014-08-01 15:00:38 EDT
(In reply to Denis Roy from comment #6)
> Looks good!  Just a nit, I think the breadcrumbs should be Home > Community
> > Planet Eclipse

Good feedback! I made an update to the patch!
Comment 8 Gunnar Wagenknecht CLA 2014-08-01 21:20:35 EDT
(In reply to Christopher Guindon from comment #5)
> https://git.eclipse.org/r/#/c/30900/

I can't access the review. It says not found or permission denied. 

Did you push it to planeteclipse.org/planeteclipse.org project in Gerrit? I didn't get the review email from Gerrit although I've configured Gerrit to do so for new reviews for 'planeteclipse.org/planeteclipse.org'.
Comment 9 Ian Skerrett CLA 2014-08-05 09:02:42 EDT
(In reply to Christopher Guindon from comment #5)
> Created attachment 245657 [details]
> Screenshot
> 
> I have a patch ready but before we go live with this, we need a new
> planeteclipse logo.
> 
> https://git.eclipse.org/r/#/c/30900/

OK, I will ask Matt to do a couple of logo concepts so we can decide which one we like.
Comment 10 Christopher Guindon CLA 2014-08-05 09:14:50 EDT
(In reply to Gunnar Wagenknecht from comment #8)
> (In reply to Christopher Guindon from comment #5)
> > https://git.eclipse.org/r/#/c/30900/
> 
> I can't access the review. It says not found or permission denied. 
> 
> Did you push it to planeteclipse.org/planeteclipse.org project in Gerrit? I
> didn't get the review email from Gerrit although I've configured Gerrit to
> do so for new reviews for 'planeteclipse.org/planeteclipse.org'.

The patch was a draft. It should be public now!
Comment 11 Ian Skerrett CLA 2014-08-05 12:28:40 EDT
Created attachment 245734 [details]
Example logos for PlanetEclipse

Here are some concepts for a new PlanetEclipse logo.  Thoughts?
Comment 12 Christopher Guindon CLA 2014-08-05 12:40:29 EDT
Created attachment 245735 [details]
Screenshot with new logo

(In reply to Ian Skerrett from comment #11)
> Created attachment 245734 [details]
> Example logos for PlanetEclipse
> 
> Here are some concepts for a new PlanetEclipse logo.  Thoughts?

I prefer the first logo. I think it's the one that would work best with the Solstice theme. 

The Solstice theme looks better with a wide logo.

Attaching an example of how it would look like with Solstice.
Comment 13 Gunnar Wagenknecht CLA 2014-08-05 13:39:26 EDT
I agree with the wider logo. One thing that came to my mind is readability of the logo. Would it be possible to have the word "planet" in front of the regular Eclipse logo so that it actually reads "planet()eclipse"? Not sure if that works from a design perspective. Just a thought.
Comment 14 Ian Skerrett CLA 2014-08-05 13:53:58 EDT
(In reply to Gunnar Wagenknecht from comment #13)
> I agree with the wider logo. One thing that came to my mind is readability
> of the logo. Would it be possible to have the word "planet" in front of the
> regular Eclipse logo so that it actually reads "planet()eclipse"? Not sure
> if that works from a design perspective. Just a thought.

Graphically I don't think it will work. Also, we seem to have done okay with the 'planet' after 'eclipse' in the existing logo so I don't think it is a big concern.
Comment 15 Christopher Guindon CLA 2014-08-07 10:30:05 EDT
Can we roll this out soon?
Comment 16 Ian Skerrett CLA 2014-08-07 10:32:18 EDT
(In reply to Christopher Guindon from comment #15)
> Can we roll this out soon?

Once we decide on the logo, the Board of Directors needs to approve it. 

Are there any other comments about the PlanetEclipse logo?
Comment 17 Denis Roy CLA 2014-08-11 09:55:20 EDT
Cool logo!
Comment 18 Ian Skerrett CLA 2014-08-21 08:31:41 EDT
Seems like there are no other discussion so I am going to get Board approval for this logo.
Comment 19 Ian Skerrett CLA 2014-08-22 14:56:17 EDT
OK, we can proceed with this logo.
Comment 20 Christopher Guindon CLA 2014-08-25 11:18:01 EDT
(In reply to Gunnar Wagenknecht from comment #8)
> (In reply to Christopher Guindon from comment #5)
> > https://git.eclipse.org/r/#/c/30900/
> 
> I can't access the review. It says not found or permission denied. 
> 
> Did you push it to planeteclipse.org/planeteclipse.org project in Gerrit? I
> didn't get the review email from Gerrit although I've configured Gerrit to
> do so for new reviews for 'planeteclipse.org/planeteclipse.org'.

Awesome! Gunnar how do you want to proceed for making this live? 

Is anyone available to +1 the new theme on Gerrit?

Do we need to do any manual steps to deploy the new theme? Do we need to copy the files anywhere or will it use the files from the git repo?
Comment 21 Gunnar Wagenknecht CLA 2014-08-25 11:51:43 EDT
I +2'ed the review. Feel free to merge it when you are ready. :)
Comment 22 Christopher Guindon CLA 2014-08-25 11:54:10 EDT
(In reply to Gunnar Wagenknecht from comment #21)
> I +2'ed the review. Feel free to merge it when you are ready. :)

thanks, Done!
Comment 23 Christopher Guindon CLA 2014-08-25 12:03:04 EDT
The new theme is live :)

I think we are done here?

http://planeteclipse.org/planet/
Comment 24 Christopher Guindon CLA 2014-08-25 12:43:28 EDT
I found an issue we should address before closing this bug

Bug 442514 - Remove faces on extra small screens (Mobile)
Comment 25 Christopher Guindon CLA 2014-08-26 15:35:52 EDT
I think we are done here. We can open new bugs to address any issues with the new theme in the future.

Thanks everyone!
Comment 26 Gunnar Wagenknecht CLA 2014-08-27 03:05:51 EDT
Thanks Chris! Great work.