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

Update pipelines, secrets, quota and membership page headers to be consistent #796

Merged
merged 1 commit into from
Nov 2, 2016

Conversation

sg00dwin
Copy link
Member

@sg00dwin sg00dwin commented Nov 2, 2016

with the rest of secondary level pages.

Fixes #779
Fixes #780

@spadgett PTAL

screen shot 2016-11-01 at 3 10 47 pm

screen shot 2016-11-01 at 3 07 40 pm

screen shot 2016-11-01 at 3 07 22 pm

screen shot 2016-11-01 at 3 07 14 pm

@sg00dwin sg00dwin changed the title Update pipelines, secrets and membership page headers to be consistent Update pipelines, secrets, quota and membership page headers to be consistent Nov 2, 2016
Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

Let me know if you need help removing the corresponding breadcrumb code in the JavaScript.

@@ -5,30 +5,32 @@
<div class="middle-container">
<div class="middle-header">
<div class="container-fluid">
<breadcrumbs breadcrumbs="breadcrumbs"></breadcrumbs>
Copy link
Member

Choose a reason for hiding this comment

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

If you remove the breadcrumbs from the view, we should also take the breadcrumbs out of the controller.

@@ -6,18 +6,19 @@
<div class="middle-container">
<div class="middle-header">
<div class="container-fluid">
<breadcrumbs breadcrumbs="breadcrumbs"></breadcrumbs>
Copy link
Member

Choose a reason for hiding this comment

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

If you remove the breadcrumbs from the view, we should also take the breadcrumbs out of the controller.

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

Disregard my earlier comments. It looks like there are no breadcrumbs set in the controller, and those lines should not have been in the view to begin with.

@spadgett
Copy link
Member

spadgett commented Nov 2, 2016

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 7897cc5

@openshift-bot
Copy link

openshift-bot commented Nov 2, 2016

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/638/) (Base Commit: 00eb12f)

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.

None yet

3 participants