Re: [PATCH 0/2] docs: make tables responsive with content-aware column widths
Jonathan Corbet <[email protected]>
| Newsgroups | org.kernel.vger.linux-doc,org.kernel.vger.linux-media |
|---|---|
| Message-ID | <[email protected]> |
"Rito Rhymes" <[email protected]> writes: >> So I will look at this in more detail in the coming days. > > Thank you, I look forward to the review. > >> this is the injection of a bunch of complexity to solve a problem that >> nobody has complained about. > > I don't believe that aligns with the reality, Jon. So I have spent a while looking at this change and playing with it. It hasn't really changed my feeling about the whole thing. You speak repeatedly about rendering on browser windows of widths less than 500px. Who uses such a window? The last Nexus 1 holdout I knew gave up years ago. I would want some sort of indication that this is an actual use case - one with unusually silent users it seems - before trying to address it. I really don't like the 40-classes hack. CSS makes us do some painful things sometimes, but still ... again, I would have to see that there is a real use case before even considering it. Just setting max-width doesn't do what you want? I will listen to what others have to say and, if there is a clamor for the inclusion of this work, I'll consider it further. But, as it stands, my impression still is that it adds a lot of machine-generated complexity that humans will have to maintain going forward for a problem that nobody has complained about. Thanks, jon