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

Fix z-index & wrong CSS #1094

Merged
merged 9 commits into from
Aug 28, 2017
Merged

Fix z-index & wrong CSS #1094

merged 9 commits into from
Aug 28, 2017

Conversation

touficbatache
Copy link
Contributor

@touficbatache touficbatache commented Aug 8, 2017

Fixes and closes #1095 #1096

@touficbatache touficbatache changed the title Format HTML code Correct wrong documentation Aug 8, 2017
@codecov-io
Copy link

codecov-io commented Aug 8, 2017

Codecov Report

Merging #1094 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1094   +/-   ##
======================================
  Coverage    99.9%   99.9%           
======================================
  Files          69      69           
  Lines        3305    3305           
  Branches      405     405           
======================================
  Hits         3302    3302           
  Misses          3       3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb61827...2a5238b. Read the comment docs.

@touficbatache touficbatache changed the title Correct wrong documentation Fix z-index & wrong CSS Aug 8, 2017
Copy link
Contributor

@lynnmercier lynnmercier left a comment

Choose a reason for hiding this comment

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

Please add a section to demos/dialog.html which shows a dialog with an MDC Button with mdc-dialog__accent.

akrdxyfhsng

@lynnmercier lynnmercier self-assigned this Aug 9, 2017
@@ -116,7 +116,7 @@ They only need to match the values set for their corresponding aria attributes.

Dialog actions use system colors by default, but you can use a contrasting color, such as the palette’s accent color, to distinguish dialog actions from dialog content. To emphasize an action from other contents, add `mdc-dialog__action` to `mdc-button` to apply accent color.

```
```html
<aside class="mdc-dialog">
<div class="mdc-dialog__surface">
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs closing </div> in example

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. Just fixed it.

@trimox
Copy link
Contributor

trimox commented Aug 16, 2017

Testing in Angular MDC..LGTM. Thanks for the fix!
image

@touficbatache
Copy link
Contributor Author

No problem, happy to be a part of the community!

@touficbatache
Copy link
Contributor Author

touficbatache commented Aug 18, 2017

@lynnjepsen Should this be merged 😃?

@lynnmercier
Copy link
Contributor

I'd still like an example of a dialog with an action button added to our demo page. Then we can merge it.

@touficbatache
Copy link
Contributor Author

@lynnjepsen Sorry, I thought I added the demo but it turns out I didn't. Just added it 😃

@lynnmercier lynnmercier merged commit 88b7167 into material-components:master Aug 28, 2017
@touficbatache
Copy link
Contributor Author

No problem :)

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.

(dialog): Wrong z-index
4 participants