-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Move IE Grid documentation into a separate file #1310
Comments
Do you want to send PR with grid area declaration inheriting? |
What a fast response 😅, |
The reason why we do not copy is that it is hard to do it for all cases. For instance: .grid {
grid-template-columns: repeat(3, 1fr);
}
@media (min-width: 900px) {
grid-template-columns: repeat(4, 1fr);
}
.grid.is-big {
grid-template-columns: repeat(5, 1fr);
}
@media (min-width: 1200px) {
grid-gap: 3.5rem 2.5rem;
// we need to copy from (min-width: 900px) and dublicate .grid.is-big
} Because IE 11 dies and the CSS community doesn’t give a proper support for JS tools, it is very unlikely that somebody will write this complex logic. Do you have an idea of how we can improve docs to make it clear that you must to duplicate |
Damn to Ie. Mmmmmm....currently also, We need to consider that for support Ie (in both autoplacement mode and no-autoplacement) things are already a little bit complicated. For that reason maybe (in addition to the CSS tricks article) It will be useful to shift the grid Ie documentation in a separate file.md with more examples and explanations about logics, properties and so on. |
@Unknow-max hm, I like the idea. Do you want to try and send a pull request? |
Well, I can give it a try (in the next 3-4 days I can Pr), but let's recap a moment. We Will cut the part related to Ie grid support for the readme MD (maybe including a link the separate doc that says "Hey, if you want to know about Grid Ie, see here", and then including in the root folder a file called "grid-ie-support.md" where we will explain a little bit better the actual support state. do you think it is okay? |
I think we can try. In contrast with other questions and topics of @Dan503 what do you think if we will move Grid IE docs to separated files with many links from main README.md to have docs focused on Grid? |
Yeah I think it's a pretty good idea. The Grid portion of autoprefixer is a huge component that isn't relevant to the rest of autoprefixer so it makes sense to have separate readme files for it. |
Ok, little update guys! |
Cool, thanks for doing this 😊 I was the one that wrote the CSS Grid in IE series on CSS Tricks and probably most of the Grid related Autoprefixer documentation. I recommend installing the Markdown All in One VS Code extension to make working on markdown files a bit easier. |
The new docs look much focused. It was a really good idea. Can we simplify example (with 2-3 grid cells, instead of 4) and use some real cases like |
Done! |
Send a PR, it will be easier for me to comment there |
Sorry, I've been really busy this week. I'll try and take a look at this over the weekend. |
@ai I had a look at the Financial Times and Yandex websites. I don't see a single scrap of CSS Grid code in any of the style sheets :/ Do we have any valid examples of major websites using IE Friendly CSS Grid code that we can link to? Can you ask on Twitter maybe? |
@Unknow-max I'm reviewing it. There is a lot of broken English in it that needs fixing. I've checked out a copy of your branch and I'm making edits to it, but it will probably take a while. Also, why did you feel the need to try and explain the I'm refocusing that section to explain what Autoprefixer is actually doing when you turn auto-placement support on. |
I'm no-where near done yet but this is what I've done so far: Dan503/Grid-documentation-update |
Thanks @Unknow-max for making a start on this 😊 |
Ok! Thank you very much!
Well when I first saw the documentation I noticed that there is'nt any good documentation about the no-autoplacement option, ok, maybe is't important as the autoplacement option, but hey, it still remain an autoprefixer option. I think that in a documentation we need to leave space for all the options in equal measure, without leaving anything just because "We primarily want developers to get used to using Autoprefixer with auto-placement turned on". If the want to use autoplacement, they will use autoplacement, if they want to use no-autoplacement, they will use no-autoplacement. This is that I think! |
Ok, no problem, I pass the baton to you! Good Luck! |
Ok, i get that. We can explain |
Good, but noticed something: |
Oh I didn't realise the Grid article on CSS Tricks was focusing on |
I haven't forgotten about this. I'm just very busy at the moment and this is a pretty low priority for me. I'll probably get around to working on this again in a few weeks. |
Also can we rename this issue "Move Grid documentation into a separate file"? The current issue name doesn't align with what the actual work for the issue is anymore. |
As discussed in postcss#1310
As mentioned [here](postcss#1310 (comment))
@Unknow-max Thanks for pointing that out. The articles were written before the autoplace option existed and I never went back to update them. I've gone through all the CSS Tricks articles now and have posted updates to parts that I felt were outdated. The CSS Tricks articles now place a lot more emphasis on the new I'll get back into updating the Autoprefixer Grid documentation either tomorrow or maybe next weekend. |
Hi everyone,
Maybe this will be a duplicate issue (I already searched in the closed issues, but I don't foud anything).
Take this piece of code (grid: no-autoplace)
The first part of the code, out of the media query is translated well with the right declarations ecc.
But the second part of the code is a little bit a problem where the final css output prefixed is:
Instead when we add a duplicated
grid-area
all work good. Why autoprefixer cannot "inherit" the grid area declaration?The text was updated successfully, but these errors were encountered: