Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Re: Additional wrapper around listings tables ?
Regarding images – a simpler solution would be to get the width and height attributes removed from images in the system and let the browser size them. We can then use the max-width rule method.
Offline
Re: Additional wrapper around listings tables ?
philwareham wrote:
Regarding images – a simpler solution would be to get the width and height attributes removed from images in the system and let the browser size them. We can then use the
max-widthrule method.
No, that won’t help due to the image being wrapped inside a table with table-layout:auto.
2 test cases (view source…), compare WebKit vs Gecko or IE
http://dev.l-c-n.com/_temp/w_image.html
http://dev.l-c-n.com/_temp/w_image2.html
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: Additional wrapper around listings tables ?
philwareham wrote:
Regarding images – a simpler solution would be to get the width and height attributes removed from images in the system and let the browser size them.
I’m throwing my big no card on this. User-experience, remember the user-experience.
The only thing removing the width and height from the images will do is jagged loading. Love me some jumping content when the browser finds out the image sizes one by one, instead of knowing them from get go. That is unless we want to make the page preloaded. Oh yeah, that’s why large, cheesy badly coded applets suck. And then there is the second option of making the sizes hard-coded and clipping/scaling the images to a grid.
Offline
Re: Additional wrapper around listings tables ?
Gocom wrote:
The only thing removing the width and height from the images will do is jagged loading.
Yeah, it was a compromised solution at best.
Offline
Re: Additional wrapper around listings tables ?
phiw13 wrote:
Fwiw, for ease of tracking, I’ve filed this as issue 167.
Stef checked my patch in rev3748.
There is now a class="txp-listtables" for this specific purpose.
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: Additional wrapper around listings tables ?
Hi Stef, you seem to have missed the table on the plugins page? No div wrapper on that one at the moment.
Offline
Re: Additional wrapper around listings tables ?
Uh, that is weird. I edited that file, why didn’t svn diff pick it up ?
I’ll generate a patch later and attach it to issue 167.
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: Additional wrapper around listings tables ?
phiw13 wrote:
Uh, that is weird. I edited that file, why didn’t
svn diffpick it up ?I’ll generate a patch later and attach it to issue 167.
Bloke, ping ? Can you check that additional patch in ?
Last edited by phiw13 (2012-04-28 02:43:25)
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline