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

Open Source branding #1403

Merged
merged 4 commits into from
Dec 7, 2017
Merged

Open Source branding #1403

merged 4 commits into from
Dec 7, 2017

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented Dec 6, 2017

Different theme for Open Source.

Also makes this the default theme/brand in place of suse-branding.

SUSE logo still used - if accepted as a CF-extension hopefully we can get permission to use the Cloud Foundry logo.

@nwmac nwmac changed the title Cf branding Open Source branding Dec 6, 2017
Copy link
Contributor

@richard-cox richard-cox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The app status 'ok/deployed' colour on the app wall hasn't changed from green to blue like the app status on the app summary page has. We also show blue for the endpoint ok/connected status colour. Think both the blue or green look ok, just need to be consistent.

Copy link
Contributor

@richard-cox richard-cox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment, otherwise LGTM

@richard-cox richard-cox merged commit c0f951e into master Dec 7, 2017
@richard-cox richard-cox deleted the cf-branding branch December 7, 2017 11:42
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.

2 participants