Skip to content

Feature/135 style apps card#152

Merged
ongk merged 11 commits into
masterfrom
feature/135_style_apps_card
Mar 24, 2016
Merged

Feature/135 style apps card#152
ongk merged 11 commits into
masterfrom
feature/135_style_apps_card

Conversation

@mwhagedorn
Copy link
Copy Markdown
Contributor

Style Applications Gallery Card to match comps. Missing some usage data as unable to determine if we can get that from the API. Also missing a yellow/green 'progress bar' type display for the same reason (not sure we can get used vs allocated)

@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

<strong>Allocated Disk Space</strong>: {{applicationGalleryCardCtrl.app.entity.disk_quota}} MB
<br/>
<strong>App Instances</strong>: {{applicationGalleryCardCtrl.app.entity.instances}}
<table style="width: 100%;" class="ng-scope">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mwhagedorn You can use add class='table' and remove the style attribute, I believe.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can use a description list instead of table here: see example in helion-ui-theme

@@ -1,7 +1,25 @@
// cloud-foundry.scss
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lines 1-18 aren't needed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The uppercasing and etc is needed to match the comp.

line-height: $hpe-unit-space * 3;
vertical-align: middle;
margin-top: $hpe-unit-space;
height: $hpe-unit-space * 3;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Any reason the indent was modified here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no.. probably the stupid editor pycharm provides

@ongk
Copy link
Copy Markdown
Contributor

ongk commented Mar 24, 2016

+1 LGTM

@wchrisjohnson
Copy link
Copy Markdown
Contributor

+1 LGTM, assuming the checks complete successfully.

@ongk ongk merged commit cad127d into master Mar 24, 2016
@ongk ongk deleted the feature/135_style_apps_card branch March 24, 2016 18:41
@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

5 similar comments
@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

@randyochoa
Copy link
Copy Markdown
Contributor

From Jenkins: There was a test failure while running Jenkins tests.

irfanhabib pushed a commit that referenced this pull request Jun 1, 2017
Relay underlying message from internal method
irfanhabib pushed a commit that referenced this pull request Jun 6, 2017
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.

4 participants