- Dillo-dev - lists.mailman3.com

Re: complex buttons and limit text width

by Johannes.Hofmann＠gmx.de

On Tue, Nov 30, 2010 at 09:15:26PM +0100, Johannes Hofmann wrote: > On Tue, Nov 30, 2010 at 07:07:40PM +0000, corvid wrote: > > Do you remember whether that was a Hard Problem, or just > > one that we didn't get around to looking into? > > I don't remember, but a 100% CPU hog sounds fixable. Haha, famous last words. I still didn't find cause. It's a redraw loop somehow. Cheers, Johannes

Dec. 20, 2010

[patch]: CSS parser cleanup and media-conditional @import.

by Johannes.Hofmann＠gmx.de

On Sat, Dec 18, 2010 at 05:26:05PM +0000, Jeremy Henty wrote: > On Sat, Dec 18, 2010 at 01:51:02PM +0100, Johannes Hofmann wrote: > > > > for some reason I thought I had already answered this... > > That's strange! The only record I have (in memory or file system) is > a reply from corvid. Maybe I'm losing track! :-) No no, it was me loosing track. I checked my sent mail and I hadn't sent the reply - I just thought I did :-)

Dec. 18, 2010

[patch]: CSS parser cleanup and media-conditional @import.

by onepoint＠starurchin.org

On Sat, Dec 18, 2010 at 01:51:02PM +0100, Johannes Hofmann wrote: > > for some reason I thought I had already answered this... That's strange! The only record I have (in memory or file system) is a reply from corvid. Maybe I'm losing track! :-) > Anyway, I already looked at it and tested it a little and it seems > fine to me. > Just go ahead and commit. Thanks, it's done. Regards, Jeremy Henty

Dec. 18, 2010

[patch]: CSS parser cleanup and media-conditional @import.

by Johannes.Hofmann＠gmx.de

Hi, On Sat, Dec 18, 2010 at 01:59:09AM +0000, Jeremy Henty wrote: > > Reposting my CSS parser cleanup patches and adding the implementation > of CSS2 media-conditional @import rules. Please comment. for some reason I thought I had already answered this... Anyway, I already looked at it and tested it a little and it seems fine to me. Just go ahead and commit. Cheers, Johannes

Dec. 18, 2010

[patch]: CSS parser cleanup and media-conditional @import.

by onepoint＠starurchin.org

Reposting my CSS parser cleanup patches and adding the implementation of CSS2 media-conditional @import rules. Please comment. Regards, Jeremy Henty

Dec. 18, 2010

patch: border-color: transparent

by Johannes.Hofmann＠gmx.de

On Tue, Dec 14, 2010 at 04:23:53PM +0000, corvid wrote: > Jorge wrote: > > Is there an open-issue/reason-not-to-commit it? > > I just wanted others to have a look at it first since I don't know > cssparser/styleengine in any detail. Looks good to me. Cheers, Johannes

Dec. 14, 2010

patch: border-color: transparent

by corvid＠lavabit.com

Jorge wrote: > Is there an open-issue/reason-not-to-commit it? I just wanted others to have a look at it first since I don't know cssparser/styleengine in any detail.

Dec. 14, 2010

patch: border-color: transparent

by jcid＠dillo.org

On Tue, Dec 14, 2010 at 02:02:08AM +0000, corvid wrote: > With user agent set to firefox, google likes to give me a > border-color: transparent, and I was tired of the no-values msg. It looks OK, according to the SPEC: <quote CSS3> 4.2.3. "transparent" color keyword CSS1 introduced the "transparent" value for the background-color property. CSS2 allowed border-color to also accept the "transparent" value. The Open eBook(tm) Publication Structure 1.0.1 [OEB101] extended the "color" property to also accept the "transparent" keyword. CSS3 extends the color value to include the "transparent" keyword to allow its use with all properties that accept a <color> value. This simplifies the definition of those properties in CSS3. transparent Fully transparent. This keyword can be considered a shorthand for transparent black, rgba(0,0,0,0), which is its computed value. </quote> Is there an open-issue/reason-not-to-commit it? -- Cheers Jorge.-

Dec. 14, 2010

[patch]: CSS parser cleanup

by onepoint＠starurchin.org

On Tue, Dec 14, 2010 at 02:10:49AM +0000, corvid wrote: > Jeremy wrote: > > diff -r 6276951c09ca -r 863857740f7f src/cssparser.cc > > --- a/src/cssparser.cc Sun Dec 05 16:45:24 2010 +0000 > > +++ b/src/cssparser.cc Sun Dec 05 18:50:21 2010 +0000 > > @@ -1422,12 +1422,7 @@ > > else if (ttype == CSS_TK_STRING) > > urlStr = dStrdup (tval); > > > > - /* Skip all tokens until the expected end. */ > > - while (!(ttype == CSS_TK_END || > > - (ttype == CSS_TK_CHAR && (tval[0] == ';')))) > > - nextToken(); > > - > > - nextToken(); > > + ignoreStatement(); > > > > if (urlStr) { > > MSG("CssParser::parseImport(): @import %s\n", urlStr); > > I just wanted to mention that CSS2 says that there can be media types > after the URL there... Next on the TO DO list! But I don't think this patch will block implementing that - just call nextToken() before ignoreStatement() and do the right thing if it's a string. Regards, Jeremy Henty

Dec. 14, 2010

[patch]: CSS parser cleanup

by corvid＠lavabit.com

Jeremy wrote: > diff -r 6276951c09ca -r 863857740f7f src/cssparser.cc > --- a/src/cssparser.cc Sun Dec 05 16:45:24 2010 +0000 > +++ b/src/cssparser.cc Sun Dec 05 18:50:21 2010 +0000 > @@ -1422,12 +1422,7 @@ > else if (ttype == CSS_TK_STRING) > urlStr = dStrdup (tval); > > - /* Skip all tokens until the expected end. */ > - while (!(ttype == CSS_TK_END || > - (ttype == CSS_TK_CHAR && (tval[0] == ';')))) > - nextToken(); > - > - nextToken(); > + ignoreStatement(); > > if (urlStr) { > MSG("CssParser::parseImport(): @import %s\n", urlStr); I just wanted to mention that CSS2 says that there can be media types after the URL there...

Dec. 14, 2010