Sure, but not when we are about to release 1.3 final. Or is this something worth delaying 1.3 for a couple of weeks? Martijn
On Dec 28, 2007 3:11 PM, Johan Compagner <[EMAIL PROTECTED]> wrote: > And thats just stupid comments in the first place... > We should always just strip those out > > On 12/28/07, Matej Knopp <[EMAIL PROTECTED]> wrote: > > But just to be fair, it's not the xml declaration itselfs that breaks > > IE (both 6 and 7). It's the apache license comment _between_ the <xml > > declaration and the doctype. > > > > -Matej > > > > On Dec 28, 2007 2:40 PM, Matej Knopp <[EMAIL PROTECTED]> wrote: > > > On Dec 28, 2007 2:24 PM, Martijn Dashorst <[EMAIL PROTECTED]> > > wrote: > > > > I am still not convinced that the xml declaration should be there in > the > > > > first place, or at least that it should have been added this late in > the > > > > game. Look at what is happening now: we are discussing new features > to > > work > > > > around one of the stupidest browsers in the world just because > someone > > (just > > > > 1 person) reported some missing xml declaration. I warned against it > and > > see > > > > the mess we're in. > > > > Revert the damned xml declarations and release 1.3 final. Pick up > the > > issue > > > > again in 1.4 and address it properly. We have been able to build and > > ship > > > > wicket applications for over 3 years without the declaration, so I > don't > > see > > > > why we can't do so another 4 months. > > > +1 > > > > > > -Matej > > > > > > > > > > > > > > Martijn > > > > > > > > > > > > > > > > On Dec 28, 2007 11:02 AM, Matej Knopp <[EMAIL PROTECTED]> wrote: > > > > > > > > > On Dec 28, 2007 2:29 AM, Juergen Donnerstag > > > > > <[EMAIL PROTECTED]> wrote: > > > > > > On Dec 28, 2007 2:15 AM, Matej Knopp <[EMAIL PROTECTED]> > wrote: > > > > > > > But how can we know that for some pages users don't want to > force > > the > > > > > > > quirks mode? > > > > > > > I'm big -1 on stripping the xml declaration for all pages by > > default. > > > > > > > > > > > > That is exactly my point. It most certainly will break existing > > > > > applications. > > > > > Agree on that. > > > > > > > > > > > > > That would break any application where the users are relying > on > > <xml > > > > > > > declaration making IE use quirks mode. > > > > > > > > > > > > I wonder how many users actually do that compared to (IMO rather > > weird > > > > > > thinking) that a proper xml document will swicth IE into quirks > mode > > > > > > (which is the old buggy render mode). How many users actually > have > > the > > > > > > intend to deliberately switch into quirks mode rather than the > other > > > > > > way around (use a std compliant mode). > > > > > > > > > > It's not that unusual really. It's a way to ensure that all > browsers > > > > > use border-box box-sizing. Since IE doesn't support the border-box > css > > > > > attribute, if you want to have sizes calculated like that you need > to > > > > > force ie to go to quirks mode. I certainly don't think we can > change > > > > > thing like this silently. And I don't even thing we have a reason > for > > > > > it. > > > > > > > > > > Right now the problem why error pages doesn't work with IE really > is > > > > > the comment between <xml declaration and doctype. I've tested it. > > > > > > > > > > > > > > > > > > > > > > > > > We should focus on the problem itself, and that is the > > > > > > > Apache header between the <?xml declaration and doctype which > is > > what > > > > > > > completely breaks IE. > > > > > > > > > > > > I'm not sure this is true. Anything, including the xml decl, > before > > > > > > the doctype makes IE switch into quirks mode. > > > > > Yes, but for our error pages we can either remove the xml > declaration > > > > > or not care that it switches IE into quirksmode. Quirksmode is not > the > > > > > real problem here. The problem is that right now IE doesn't show > the > > > > > error pages at all, quirksmode or not. > > > > > > > > > > > > > > > > > > > > Now I can see two simple solution. Either wraps the header > comment > > > > > > > with <wicket:remove> or move the header comment after doctype. > > > > > > > > > > > > I'm not sure this will work. Did you try it already? > > > > > Yes, I've tried both. They both work and IE shows the pages > properly. > > > > > > > > > > If we really want the behavior when you add an <xml header> to > file > > > > > and then don't want to show it in ouput, we should have a way to > > > > > configure that per file, such as <wicket:stripXmlHeader/> > somewhere in > > > > > the markup. > > > > > > > > > > -Matej > > > > > -Matej > > > > > > > > > > > > Juergen > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > Buy Wicket in Action: http://manning.com/dashorst > > > > Apache Wicket 1.3.0-rc2 is released > > > > Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-rc1/ > > > > > > > > > > -- Buy Wicket in Action: http://manning.com/dashorst Apache Wicket 1.3.0-rc2 is released Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-rc1/
