Skip Menu |

This queue is for tickets about the CSS-Inliner CPAN distribution.

Report information
The Basics
Id: 85130
Status: resolved
Priority: 0/
Queue: CSS-Inliner

People
Owner: Nobody in particular
Requestors: spt [...] jobindex.dk
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 3674
Fixed in: (no value)



Subject: Support for :first-child and :last-child
HTML::Query was patched to support the :first-child and :last-child pseudoclasses. It would therefore be nice with support for them in CSS::Inliner as well. Attached is a patch that enables the use of these.
Subject: css-inliner-first-child.patch
--- Inliner.pm 2013-05-08 09:26:32.636465144 +0200 +++ Inliner.pm 2013-05-08 09:26:57.160465928 +0200 @@ -282,7 +282,7 @@ my $properties = $$entry{properties}; #skip over psuedo selectors, they are not mappable the same - if ($selector =~ /[\w\*]:(?:(active|focus|hover|link|visited|after|before|selection|target|first-line|first-letter|first-child|first-child))\b/io) { + if ($selector =~ /[\w\*]:(?:(active|focus|hover|link|visited|after|before|selection|target|first-line|first-letter))\b/io) { $self->_report_warning({ info => "The pseudo-class ':$1' cannot be supported inline" }); next; }
Merged into github with: https://github.com/kamelkev/CSS-Inliner/commit/3e0210a9da36e6f839b82961e4bdd641b4e1df0c I noticed the typo whereby I typed first-child twice... need more eyes in this code! Thanks for the contributions, as noted prior I will provide a release via cpan once I resolve the versioning issue that became present once I moved to github. thanks, Kevin On Wed May 08 03:33:18 2013, spaaske wrote: Show quoted text
> HTML::Query was patched to support the :first-child and :last-child > pseudoclasses. It would therefore be nice with support for them in > CSS::Inliner as well. > > Attached is a patch that enables the use of these.