Does not match strings as expected

[debanjanbasu]

The RCS is breaking while there are certain code blocks inside the html, such as comments to target IE specific code.

[JPeer264]

Can you provide a sample where it does not work? Is there also any error?

[debanjanbasu]

Code sample

```html

Before the Bell Friday, 4th March, 2016 Strap 5l-5l SPI up 8 points at 5382 AUD at US74.73 cents Dow Jones +0.4% S&P500 +0.3% Nasdaq +0.3% FTSE100 +0.3% CAC -0.6% DAX -0.7% Spot gold +1.4% at $US1261.92 an ounce Brent oil +2.1% at $US52.54 a barrel Iron ore +0.04% at $US52.56 a tonne Strap 5l-5l CAC -0.6% DAX -0.7% Spot gold +1.4% at $US1261.92 an ounce Brent oil +2.1% at $US52.54 a barrel Iron ore +0.04% at $US52.56 a tonne Strap 1s-3l Did I leave the baby in the car? Esther Han The mum who let five strangers breastfeed her baby Mum's loving kiss leaves baby fighting for life Esther Han Mum's loving kiss leaves baby fighting for life Strap 1s-1s Evans: Why Clarkson had to go Esther Han How can we trust Tara Brown now? Esther Han Strap x Which airline wins crossing the pond? Esther Han Which airline wins crossing the pond? Esther Han Which airline wins crossing the pond? Esther Han Strap xl Duncan Jones steps out of dad David Bowie's shadow with Warcraft This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Duncan Jones steps out of dad David Bowie's shadow with Warcraft This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Duncan Jones steps out of dad David Bowie's shadow with Warcraft This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Strap 1s-2 Did I leave the baby in the car? Esther Han Which airline wins crossing the pond? Esther Han Which airline wins crossing the pond? Esther Han Strap 1s-1s_1s-1s Mary Poppins to get a sequel, starring Emily Blunt Darren Kane How can we trust Tara Brown now? Evans: Why Clarkson had to go How can we trust Tara Brown now? Strap 1-1_1-1 Woman tries to name her baby 'Cyanide' Smiling new born babies Trends only girls of Bo's would understand Favourite quotes from the Jungle Book Strap 1s-2l Did I leave the baby in the car? Esther Han The mum who let five strangers breastfeed her baby This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Mum's loving kiss leaves baby fighting for life This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Esther Han Sponsored John, Jake and Harry: the who's who of Taylor Swift breakup songs This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. John, Jake and Harry: the who's who of Taylor Swift breakup songs This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. John, Jake and Harry: the who's who of Taylor Swift breakup songs This is optional wof text, configurable on/off in CQ author, and usually turned on for the larger stacked stories. Follow us connect with us contact us my account afr This email is sent by Fairfax Media Limited. ABN. 15 008 663 161. 1 Darling Island Rd Pyrmont, NSW, 2009, Australia. You have received this email because our records show you are a subscriber to the AFR. Your email adress is: %%email_address%% If you feel you received this email by mistake or no longer wish to receive the “Name newsletter”, simply unsubscribe here. Note: This may take up to 48 hours and will only discontinue the “Name newsletter” and does not cancel other email updates or your subscription. Please read Fairfax Media Limited’s Privacy policy. “Name newsletter” contains general information, does not provide stock recommendations, financial or investment advice or gaurantee returns and has been generated without taking into account your objectives, financial situation or needs. You should seek independent advice before acting on any information or making any investment decisions.

```

This is the sample HTML file. All the conditional HTML code is causing the following code to be ignored. There is no error while compiling though. It is just not rewriting the classes and the ids in the HTML.

[JPeer264]

Well yeah, I figured that there is a small bug in my regex. Actually it should just trigger everything between " " and ' '. As it is now it will also start at words like doesn't or it's. A quick fix for you might be to encode all ' as ' and " as " in your plain text or comments.

I will give you an update as soon as I solved this problem.

[JPeer264]

Please try it again with v0.0.3

[debanjanbasu]

v0.0.3 seems to not compile the CSS at all. This is my grunt config :

rcs: {
    options: {},
    css: {
        options: {
            replaceCss: true
        },
        files: [{
            expand: true,
            cwd: 'dev/',
            src: 'csso/*.css',
            dest: 'rcs/',
        }]
    },
    all: {
        files: [{
            expand: true,
            cwd: 'dev/',
            src: ['*.html'],
            dest: 'rcs/',
        }]
    }
}

[JPeer264]

Hm weird. I made now a project with your config where all CSS files in ./dev/csso and all HTML files in ./dev got compiled.

Please also make sure that rcs:css is loaded before rcs:all. Try following for me:

1. Delete the rcs folder
2. Make sure your paths in the grunt config are correct
3. Run in your shell grunt rcs:css rcs:all

What is the output now?

[debanjanbasu]

It worked 👍 The trick is to ensure that the input CSS is not uglified (in one line). If the sass-->css is compact, it won't work. It has to be 'compressed'. I.e. CSS selector in it's own line.

[JPeer264]

I am glad that it worked.

Oh that is a good hint. I will put this into the readme. Thanks!