Skip to content
This repository has been archived by the owner on Feb 9, 2022. It is now read-only.

use CardGrid in AppList #95

Merged
merged 2 commits into from
Feb 20, 2018
Merged

Conversation

prydonius
Copy link
Contributor

@prydonius prydonius commented Feb 20, 2018

depends on #92 and should be rebased once #92 is merged

screen shot 2018-02-20 at 15 01 55

Copy link
Contributor

@Angelmmiguel Angelmmiguel left a comment

Choose a reason for hiding this comment

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

LGTM. Minor change :)

@@ -20,17 +21,24 @@ class AppList extends React.Component<IAppListProps> {
return (
<section className="AppList">
<header className="AppList__header">
<h1>Applications</h1>
<div className="row padding-t-big">
Copy link
Contributor

Choose a reason for hiding this comment

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

Add the align-center class to center the button vertically and a collapse-* class to make it responsive

@Angelmmiguel
Copy link
Contributor

Still LGTM!

@Angelmmiguel Angelmmiguel merged commit 03fc5fb into vmware-archive:master Feb 20, 2018
@prydonius prydonius deleted the applist-cards branch February 20, 2018 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants