Skip to content
This repository has been archived by the owner on Nov 6, 2024. It is now read-only.

Add card image tag and horizontal card #333

Merged
merged 4 commits into from
Aug 9, 2018
Merged

Add card image tag and horizontal card #333

merged 4 commits into from
Aug 9, 2018

Conversation

scote
Copy link
Contributor

@scote scote commented Jul 9, 2018

✔️ Ready to merge

  • Card image
  • Card image floating FAB
  • Horizaton card

@scote scote changed the title Add card image tag Add card image tag and horizontal card Jul 9, 2018
@scote
Copy link
Contributor Author

scote commented Jul 25, 2018

That's weird on my machine I have several linter errors when running yarn lint:lib. CircleCi does not seem to see those one.

Edit : I saw the errors in XML file created from Circle CI.

@jfcere
Copy link
Contributor

jfcere commented Jul 25, 2018

Can you give an example of those weird errors?

@scote
Copy link
Contributor Author

scote commented Jul 30, 2018

I juste run the linter and I only get warnings for Renderer service this time. So it's ready to review.

<div class="section">

<h5 class="light">HTML Structure</h5>

<app-code-snippet>
&lt;mz-card
class="blue-grey darken-1 white-text"
[horizontal]=="true"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove =

@@ -12,6 +12,12 @@ import { IPropertyRow } from '../shared/properties-table/properties-table.compon
})
export class CardComponent {
properties: IPropertyRow[] = [
{ name: 'horizontal',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not align correctly

}

private hasImageTitleTagAndNotEmpty(): boolean {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove

button[mzHalfwayFab]`,
})
export class MzHalfwayFabDirective {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove line

@scote scote self-assigned this Aug 9, 2018
@scote scote merged commit 610f2e0 into master Aug 9, 2018
@scote scote deleted the card branch August 9, 2018 02:32
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