Bug 1320238

Summary:	Branding: Foreman branding CSS used for Puppet Class Edit dialog
Product:	Red Hat Satellite	Reporter:	Roman Plevka <rplevka>
Component:	WebUI	Assignee:	Shimon Shtein <sshtein>
WebUI sub component:	Foreman	QA Contact:	Katello QA List <katello-qa-list>
Status:	CLOSED NOTABUG	Docs Contact:
Severity:	high
Priority:	unspecified	CC:	bbuckingham, kybaker, mmccune, rplevka
Version:	6.2.0	Keywords:	Regression, Triaged
Target Milestone:	Unspecified	Flags:	kybaker: needinfo-
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-04-05 23:04:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Roman Plevka 2016-03-22 15:51:36 UTC

Description of problem:
Configure -> Classes -> (tabs) Smart Class Parameter && Smart Variables
use the (blueish) foreman CSS 

Version-Release number of selected component (if applicable):
6.2 Snap4

How reproducible:
always

Steps to Reproduce:
1.navigate to Configure -> Classes -> (tabs) Smart Class Parameter && Smart Variables

Actual results:
Foreman branding uses

Expected results:
RH branding used

Additional info:

Comment 2 Shimon Shtein 2016-03-23 09:06:31 UTC

You are right, it looks quite blue.
What do you think should be the correct color theme for this page?
The colors did not change from SAT6.1 though...

Comment 3 Roman Plevka 2016-03-23 14:52:48 UTC

(In reply to Shimon Shtein from comment #2)
> You are right, it looks quite blue.
> What do you think should be the correct color theme for this page?
> The colors did not change from SAT6.1 though...

Sorry, had a closer look, it really looks like there were no different colors in the previous versions. All those big buttons and blue highlights gave me instant 'foreman' feeling.
I think we can close this as NOTABUG if there's really no missed satellite CSS.

Comment 4 Shimon Shtein 2016-03-28 13:54:08 UTC

Kyle, what do you think about this screen?
Right now it inherits its style from patternfly AFAIK.

Comment 5 Mike McCune 2016-04-05 23:04:15 UTC

I vote CLOSED:NOTABUG, feel free to re-open if strong feelings otherwise.