Skip to content

Before after and focus#37

Open
jasonfox wants to merge 5 commits into
premasagar:masterfrom
jasonfox:before-after-and-focus
Open

Before after and focus#37
jasonfox wants to merge 5 commits into
premasagar:masterfrom
jasonfox:before-after-and-focus

Conversation

@jasonfox

@jasonfox jasonfox commented Jun 5, 2018

Copy link
Copy Markdown

Needed to clear before and after for all elements as well as link hover, active, etc. pseudo classes.

@ddelrio1986

Copy link
Copy Markdown

It seems like many selectors were removed from different places instead of just new stuff added. Why were they removed?

Also seems to be a few lines with extra whitespace added at the end.

@jasonfox

jasonfox commented Jun 7, 2018

Copy link
Copy Markdown
Author

I don't think that I removed any selectors, though, I may have inserted some into the middle of a line. Do you have an example of something that was removed? As for the whitespaces, yeah, that was my bad.

@seriousben

Copy link
Copy Markdown

This would be a great addition.

@ddelrio1986 ddelrio1986 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Here are some differences I found. Maybe I'm looking at this wrong?

Comment thread cleanslate.css

/* HTML5 - Sept 2013 taken from MDN https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/HTML5/HTML5_element_list */
.cleanslate article, .cleanslate aside, .cleanslate figure, .cleanslate figcaption, .cleanslate footer, .cleanslate header, .cleanslate menu, .cleanslate nav, .cleanslate section, .cleanslate time, .cleanslate mark, .cleanslate audio, .cleanslate video, .cleanslate abbr, .cleanslate address, .cleanslate area, .cleanslate blockquote, .cleanslate canvas, .cleanslate caption, .cleanslate cite, .cleanslate code, .cleanslate colgroup, .cleanslate col, .cleanslate datalist, .cleanslate fieldset, .cleanslate main, .cleanslate map, .cleanslate meta, .cleanslate optgroup, .cleanslate output, .cleanslate progress, .cleanslate svg {
.cleanslate article, .cleanslate aside, .cleanslate figure, .cleanslate figcaption, .cleanslate footer, .cleanslate header, .cleanslate menu, .cleanslate nav, .cleanslate section, .cleanslate time, .cleanslate mark, .cleanslate audio, .cleanslate video, .cleanslate abbr, .cleanslate address, .cleanslate area, .cleanslate blockquote, .cleanslate canvas, .cleanslate caption, .cleanslate cite, .cleanslate code, .cleanslate colgroup, .cleanslate col, .cleanslate datalist, .cleanslate fieldset, .cleanslate main, .cleanslate map, .cleanslate meta, .cleanslate optgroup, .cleanslate output, .cleanslate progress, .cleanslate *:before, .cleanslate *:after {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

svg seems to be removed here

Comment thread cleanslate.css
font-style:italic !important;
}
.cleanslate kbd, .cleanslate samp, .cleanslate code, .cleanslate pre {
.cleanslate kbd, .cleanslate samp, .cleanslate code {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

pre was removed here.

Comment thread cleanslate.css
color: #529 !important;
}
.cleanslate a, .cleanslate a *, .cleanslate input[type=submit], .cleanslate input[type=button], .cleanslate input[type=radio], .cleanslate input[type=checkbox], .cleanslate select, .cleanslate button {
.cleanslate a, .cleanslate a *, .cleanslate a:hover, .cleanslate a:focus, .cleanslate input[type=submit], .cleanslate input[type=radio], .cleanslate input[type=checkbox], .cleanslate select {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

.cleanslate input[type=button] and .cleanslate button was removed here.

Comment thread cleanslate.css
.cleanslate input[type=radio] {
margin: 2px 2px 3px 2px !important;
}
.cleanslate input[type=submit]:active, .cleanslate button:active {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

.cleanslate input[type=button]:active and .cleanslate button:active were removed here.

Comment thread cleanslate.css
text-decoration:none !important;
list-style-type:disc !important;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

another one removed here.

@premasagar

Copy link
Copy Markdown
Owner

Thanks for the pull request. Yes it looks like there were a couple of accidental deletions. And it looks like the version number has been changed. If you could fix those, I'd be happy to merge.

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