Some kind of core CSS is likely to be needed, considering that ALL inline styles should go not just hidden/display:none.
The special class is what I originally suggested. For me personally the approach with css class would be easier to live with. Another option to consider could be an inline <style> injected into the heder. Unlike inline style attribute this can be nonced. Can't tell it's better than predefined file, just hope this can trigger an idea in someone's head. чт, 16 янв. 2020 г. в 08:20, Emond Papegaaij <emond.papega...@gmail.com>: > > > IMHO we have to decide first whether we want to use the "hidden" > attribute. > > > I'm not sure about the pros and cons at the moment. > > > > > > Depending on that decision (NO) we'll have to write CSS classes anyway > > > or (YES) we might want to add them additionally. > > > In the latter case I'm not sure we need a CSS file at all, just to > > > contain "display: none". > > > > > > > Right! > > This is how this discussion started (question 1): if we use 'hidden' > > attribute for this case then there is no need of CSS file. > > For the other cases/tickets we can decide separately. > > I disagree on this point. We use display: flex quite a lot in our > application. This will render the hidden attribute useless. The hidden > attribute without additional styling will break our application (and > therefore probably others as well). Whatever we do, we should make > sure the tags stay hidden in most cases (we can't protect against very > specific styling with !important). > > I'm ok with [data-wicket-placeholder][hidden] as a rule, but I don't > see the benefit over class="wicket--hidden". > > Also, we keep on focusing on this particular use case, but there are > others. How about this in Form: > > style="width:0px;height:0px;position:absolute;left:-100px;top:-100px;overflow:hidden". > I'm not sure this can be replaced by hidden, maybe it can. > > Best regards, > Emond > > > > > > > Thanks > > > Sven > > > > > > > > > On 15.01.20 18:52, Martin Grigorov wrote: > > > > On Wed, Jan 15, 2020, 19:29 Sven Meier <s...@meiers.net> wrote: > > > > > > > >> I disagree, "hidden" has the perfect semantic for what we're doing > with > > > >> placeholders or other <div>s. > > > >> > > > > How about such CSS rule: > > > > > > > > [data-wicket-placeholder][hidden] > > > > { > > > > display : none !important; > > > > } > > > > > > > > ? > > > > > > > > > > > >> Sven > > > >> > > > >> > > > >> On 15.01.20 11:44, Emond Papegaaij wrote: > > > >>>>> According to my tests > > > >>>>> textarea with hidden attribute works the same way as `display: > none` > > > >>>>> (latest Chrome, FF, Chromium) > > > >>>>> > > > >>> Maxim and Andrew: Yes, the display: none is very important. (That's > > > >>> why it even has !important). The hidden attribute has the lowest > > > >>> priority possible. Any matching CSS rule will override it's > behavior. > > > >>> For example, if you have this html: > > > >>> <html> > > > >>> <style>li { display: list-item; }</style> > > > >>> <ul><li hidden>This will be visible</li></ul> > > > >>> </html> > > > >>> The li is visible, even though it has the hidden attribute. This is > > > >>> not what we want. Keep in mind that this is about components with > > > >>> isVisible() == false. The only reason we render something is > because a > > > >>> placeholder is needed for Ajax updates. > > > >>> > > > >>>> Additionally, I think we should not use just > > > >>>> [hidden] { > > > >>>> ... > > > >>>> } > > > >>>> > > > >>>> in wicker-core.css because this will affect *all* elements in the > > > page. > > > >> And > > > >>>> the applications may not want Wicket styling for their own hidden > > > >> elements. > > > >>>> The CSS rule should be namespaced with "wicket-something", like > > > >>>> wicket-xyz[hidden] {...} > > > >>> I totally agree with Martin here. hidden already has semantics, we > > > >>> shouldn't tamper with that. Adding 'display: none!important' to > hidden > > > >>> breaks the behavior described above. > > > >>> > > > >>>>> So I would vote for using `hidden="hidden"` for placeholders and > > > >> preserving > > > >>>>> `data-wicket-placeholder` > > > >>>>> (due to hidden can be used in regular markup while > > > >>>>> `data-wicket-placeholder` is clear way to determine the > placeholders) > > > >>> The data-wicket-placeholder is indeed needed. All placeholders are > > > >>> hidden, but not all hidden elements are placeholders. As stated > above, > > > >>> I'd vote against using the hidden attribute. We need something > with a > > > >>> wicket namespace here. We can have the CSS match > > > >>> [data-wicket-placeholder], but I'm not a big fan of matching data > > > >>> attributes in CSS. > > > >>> > > > >>> Best regards, > > > >>> Emond > > > >>> > > > >>>>> On Wed, 15 Jan 2020 at 04:55, Andrew Kondratev < > and...@kondratev.pro > > > > > > > >>>>> wrote: > > > >>>>> > > > >>>>>> Hi Everyone! > > > >>>>>> > > > >>>>>> Is this "none !important" actually needed? Elements with hidden > > > >> attribute > > > >>>>>> are hidden by themselves, except the cases when they have some > > > display > > > >>>>>> overrides (see > > > >>>>>> > > > >> > > > > https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/hidden > > > >>>>>> ). > > > >>>>>> If developer/user does override he or she could take care of > this by > > > >>>>>> himself. > > > >>>>>> > > > >>>>>> If the only style in core css is this wicket--hidden the whole > css > > > is, > > > >>>>>> probably unnecessary. > > > >>>>>> > > > >>>>>> ср, 15 янв. 2020 г. в 10:38, Sven Meier <s...@meiers.net>: > > > >>>>>> > > > >>>>>>> Hi Edmond, > > > >>>>>>> > > > >>>>>>> then I'll ask again: > > > >>>>>>> > > > >>>>>>> Why not render a "hidden" attribute and style it in the core > css: > > > >>>>>>> > > > >>>>>>> |[hidden] { display: none !important}| > > > >>>>>>> > > > >>>>>>> We could get rid of the "data-wicket-placeholder" attribute as > > > well, > > > >>>>> and > > > >>>>>>> let wicket-ajax check on the "hidden" attribute instead. > > > >>>>>>> > > > >>>>>>> Sven > > > >>>>>>> > > > >>>>>>> On 14.01.20 21:06, Emond Papegaaij wrote: > > > >>>>>>>>> >Using the hidden attribute is no option either, > > > >>>>>>>>> >because it is overridden by any css that sets display. > > > >>>>>>>>> > > > >>>>>>>>> why is that a no-option, hidden elements can be styled too: > > > >>>>>>>> Dropping the styling altogether will force our users to > package > > > >> their > > > >>>>>>>> own styling for artifacts generated by Wicket. These styles > are > > > not > > > >>>>> to > > > >>>>>>>> make your application look nice, but to make certain Wicket > > > >> internals > > > >>>>>>>> work as expected. In the case of wicket--hidden, not much > else can > > > >> be > > > >>>>>>>> done than use 'display:none'. In the case of the Form parts, > more > > > >>>>>>>> elaborate styling is need to keep the extra fields hidden > without > > > >>>>>>>> using display:none. Why put the burden on the user of Wicket > when > > > we > > > >>>>>>>> can package those styling ourselves? I can put the > registration of > > > >>>>> the > > > >>>>>>>> IHeaderContributor in a protected method in WebApplication. > That > > > way > > > >>>>>>>> you can change the behavior in a subclass. You can already > change > > > >> the > > > >>>>>>>> stylesheet if you need to. > > > >>>>>>>> > > > >>>>>>>>> Do you mean FormComponentFeedbackBorder's "color:red;" style? > > > >>>>>>>>> I wouldn't want that to be located in a 'kitchen-sink' CSS > file > > > in > > > >>>>> the > > > >>>>>>>>> future. > > > >>>>>>>> Yes, that's the styling I meant. There are more examples. IMHO > > > those > > > >>>>>>>> are typical candidates for CSS classes, but without styling > > > packaged > > > >>>>>>>> with Wicket (like with FeedbackPanel). The styling is not > going to > > > >>>>>>>> match the users application anyway. Who would want to use red > as a > > > >>>>>>>> color for a border? Even for me, being colorblind, that color > is > > > way > > > >>>>>>>> too saturated. > > > >>>>>>>> > > > >>>>>>>> Best regards, > > > >>>>>>>> Emond > > > >>>>>>>> > > > >>>>>>>>> Have fun > > > >>>>>>>>> Sven|| > > > >>>>>>>>> || > > > >>>>>>>>> > > > >>>>>>>>> > > > >>>>>>>>> On 14.01.20 20:31, Emond Papegaaij wrote: > > > >>>>>>>>>> Hi, > > > >>>>>>>>>> > > > >>>>>>>>>> Sven, we do need styling to hide components. > > > >>>>>>>>>> Component.renderPlaceholderTag renders an empty tag for > whatever > > > >> it > > > >>>>>>>>>> reads from your markup. That fact that it is empty, does not > > > mean > > > >>>>> it > > > >>>>>>>>>> is invisible. For example, in my application I had empty > > > >> list-items > > > >>>>>>>>>> (li) popping up all over the place. Using the hidden > attribute > > > is > > > >>>>> no > > > >>>>>>>>>> option either, because it is overridden by any css that sets > > > >>>>> display. > > > >>>>>>>>>> We need something that takes precedence over most styling > and > > > >> hides > > > >>>>>>>>>> the component for sure. > > > >>>>>>>>>> > > > >>>>>>>>>> Maxim, I didn't know about the application wide > > > >>>>> IHeaderContributors. > > > >>>>>>>>>> I'll give that a try, it seems cleaner to me than overriding > > > >>>>>>>>>> renderHead in Page. > > > >>>>>>>>>> > > > >>>>>>>>>> About the contents of the stylesheet, it will probably not > > > contain > > > >>>>>>>>>> much styling, but it will be more than just wicket--hidden. > For > > > >>>>>>>>>> example, the styling for the form submit handling markup > needs > > > to > > > >>>>> go > > > >>>>>>>>>> there as well. I also found some other places with inline > > > styling, > > > >>>>>>>>>> like FormComponentFeedbackBorder. We need to decide what to > do > > > >> with > > > >>>>>>>>>> those as well. > > > >>>>>>>>>> > > > >>>>>>>>>> Best regards, > > > >>>>>>>>>> Emond > > > >>>>>>>>>> > > > >>>>>>>>>> On Tue, Jan 14, 2020 at 7:19 PM Sven Meier <s...@meiers.net > > > > > >>>>> wrote: > > > >>>>>>>>>>> Hi Maxim, > > > >>>>>>>>>>> > > > >>>>>>>>>>> an empty div with hidden inputs does not take any space. > > > >>>>>>>>>>> > > > >>>>>>>>>>> Try it with wicket-examples. > > > >>>>>>>>>>> > > > >>>>>>>>>>> Have fun > > > >>>>>>>>>>> Sven > > > >>>>>>>>>>> > > > >>>>>>>>>>> Am 14. Januar 2020 18:07:23 MEZ schrieb Maxim Solodovnik < > > > >>>>>>> solomax...@gmail.com>: > > > >>>>>>>>>>>> I thought the main idea is to keep placeholder for any > element > > > >>>>> both > > > >>>>>>>>>>>> invisible and "take no space" > > > >>>>>>>>>>>> hidden attribute might work instead of `display: none` > > > >>>>>>>>>>>> The latter also hides all descendants [1], so I guess it > > > should > > > >>>>> be > > > >>>>>>>>>>>> preferable > > > >>>>>>>>>>>> > > > >>>>>>>>>>>> [1] > https://developer.mozilla.org/en-US/docs/Web/CSS/display > > > >>>>>>>>>>>> > > > >>>>>>>>>>>> On Wed, 15 Jan 2020 at 00:01, Sven Meier <s...@meiers.net > > > > > >>>>> wrote: > > > >>>>>>>>>>>>> Hi, > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> do we really need this CSS? > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> AFAIK the 'hidden' markup from Form and Component > > > placeholders > > > >>>>> do > > > >>>>>>> not > > > >>>>>>>>>>>>> need any styling actually, they look fine without it. > > > >>>>>>>>>>>>> We should add a CSS class to the markup of course, but > leave > > > >> its > > > >>>>>>>>>>>> styling > > > >>>>>>>>>>>>> to each project. > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> Cases in wicket-examples (UploadProgressBar, deprecated > > > >>>>>> ModalWindow > > > >>>>>>>>>>>> and > > > >>>>>>>>>>>>> AjaxIndicatorAppender) could switch to a different > solution > > > >>>>> (e.g. > > > >>>>>>>>>>>> their > > > >>>>>>>>>>>>> own CSS). > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> If wicket-base.css contains ".wicket--hidden" only, the > > > effort > > > >>>>>> isn't > > > >>>>>>>>>>>>> worth it IMHO. > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> Sven > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> > > > >>>>>>>>>>>>> On 14.01.20 16:23, Maxim Solodovnik wrote: > > > >>>>>>>>>>>>>> For example it can be added as > > > >>>>>>>>>>>>>> `Application.get().getHeaderContributorListeners()` > > > >>>>>>>>>>>>>> > > > >>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 16:03, Maxim Solodovnik > > > >>>>>>>>>>>> <solomax...@gmail.com> > > > >>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>> `!important` is not the silver bullet (as well as > inline > > > >>>>> style) > > > >>>>>>>>>>>>>>> `renderHead` is not as important as `onConfigure`, so I > > > >>>>> believe > > > >>>>>> it > > > >>>>>>>>>>>>>>> shouldn't be made mandatory > > > >>>>>>>>>>>>>>> Maybe there is some `hackish` way to inject this css > only > > > >> once > > > >>>>>> for > > > >>>>>>>>>>>> any > > > >>>>>>>>>>>>>>> component hierarchy? > > > >>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 15:55, Emond Papegaaij < > > > >>>>>>>>>>>>> emond.papega...@gmail.com> > > > >>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>> Rendering components without a page will indeed > require > > > you > > > >>>>> to > > > >>>>>>>>>>>> include > > > >>>>>>>>>>>>>>>> the core css file yourself. I think that's better than > > > >> adding > > > >>>>>> the > > > >>>>>>>>>>>> css > > > >>>>>>>>>>>>>>>> file with every component, as that will impose a > massive > > > >>>>>>>>>>>> overhead. > > > >>>>>>>>>>>>>>>> I've renamed the css file to wicket-core.css as > suggested > > > by > > > >>>>>>>>>>>> Martin. > > > >>>>>>>>>>>>>>>> The idea is to collect all styling used with > wicket-core > > > in > > > >>>>>> this > > > >>>>>>>>>>>> css > > > >>>>>>>>>>>>>>>> file. I do not like the idea to use a behavior for > adding > > > >> the > > > >>>>>>>>>>>>>>>> stylesheet, as that will increase the size of every > page. > > > >>>>>> Maybe a > > > >>>>>>>>>>>>>>>> temporary behavior can be used, but that will need to > be > > > >>>>>> re-added > > > >>>>>>>>>>>> on > > > >>>>>>>>>>>>>>>> every render then. Maybe, we can add a check to > > > >>>>>> super.renderHead, > > > >>>>>>>>>>>> like > > > >>>>>>>>>>>>>>>> we do with onConfgure and onInitialize? > > > >>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>> Using the hidden attribute is not a good idea, for the > > > same > > > >>>>>>>>>>>> reason as > > > >>>>>>>>>>>>>>>> I chose to use !important in the css file: any > matching > > > css > > > >>>>>> rule > > > >>>>>>>>>>>> that > > > >>>>>>>>>>>>>>>> is more specify will otherwise override the > visibility and > > > >>>>>> cause > > > >>>>>>>>>>>> the > > > >>>>>>>>>>>>>>>> component to be visible anyway (like display: flex). > > > >>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>> Best regards, > > > >>>>>>>>>>>>>>>> Emond > > > >>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>> On Tue, Jan 14, 2020 at 9:10 AM Maxim Solodovnik > > > >>>>>>>>>>>> <solomax...@gmail.com > > > >>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>> The problem as I see it > > > >>>>>>>>>>>>>>>>> The component will be rendered without page (and > without > > > >> CSS > > > >>>>>>>>>>>> file > > > >>>>>>>>>>>>>>>> itself) > > > >>>>>>>>>>>>>>>>> so element with this class will actually be visible > .... > > > >>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 15:08, Martin Grigorov > > > >>>>>>>>>>>> <mgrigo...@apache.org> > > > >>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>> On Tue, Jan 14, 2020 at 10:01 AM Maxim Solodovnik < > > > >>>>>>>>>>>>>>>> solomax...@gmail.com> > > > >>>>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>> An related question: > > > >>>>>>>>>>>>>>>>>>> Will this code > > > >>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>> > > > >> > > > > `org.apache.wicket.core.util.string.ComponentRenderer.renderComponent(Component)` > > > >>>>>>>>>>>>>>>>>>> work as expected? > > > >>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>> This method will render the component with > > > >>>>>>>>>>>> class="wicket--hidden" on > > > >>>>>>>>>>>>>>>> its > > > >>>>>>>>>>>>>>>>>> HTML element. With TagTester you can verify > > > >>>>>>>>>>>>>>>>>> CSP checks are done only by the browsers. > > > >>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 14:15, Maxim Solodovnik < > > > >>>>>>>>>>>>>>>> solomax...@gmail.com> > > > >>>>>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>> Is this comment make sense: > > > >>>>>>>>>>>>>>>>>>>> > > > >> > > > > https://github.com/apache/wicket/commit/6d91a6a9e5c1d955a53571f9fb0f76262ac5c5d2#r36784645 > > > >>>>>>>>>>>>>>>>>>>> ? > > > >>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 14:13, Martin Grigorov < > > > >>>>>>>>>>>>>>>> mgrigo...@apache.org> > > > >>>>>>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>> On Tue, Jan 14, 2020 at 9:05 AM Maxim Solodovnik > < > > > >>>>>>>>>>>>>>>>>> solomax...@gmail.com> > > > >>>>>>>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>> Wasn't aware of `hidden` attribute > > > >>>>>>>>>>>>>>>>>>>>>> (and it seems to be widely supported > > > >>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/hidden > > > >>>>>>>>>>>>>>>>>>>>>> ) > > > >>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>> Thanks for the reference, Maxim! > > > >>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>> It says "elements that are descendants of a > hidden > > > >>>>> element > > > >>>>>>>>>>>> are > > > >>>>>>>>>>>>>>>> still > > > >>>>>>>>>>>>>>>>>>>>> active, which means that script elements can > still > > > >>>>> execute > > > >>>>>>>>>>>> and > > > >>>>>>>>>>>>>>>> form > > > >>>>>>>>>>>>>>>>>>>>> elements can still submit" and this is different > than > > > >>>>>>>>>>>> display: > > > >>>>>>>>>>>>>>>> none. > > > >>>>>>>>>>>>>>>>>>>>> So we should stick with CSS display! > > > >>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>> Thanks :) > > > >>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>> On Tue, 14 Jan 2020 at 14:01, Martin Grigorov < > > > >>>>>>>>>>>>>>>> mgrigo...@apache.org > > > >>>>>>>>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>> Hi, > > > >>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>> 1) I see that such CSS resource might be used > for > > > >>>>> other > > > >>>>>>>>>>>>>>>> needs, not > > > >>>>>>>>>>>>>>>>>>>>> just > > > >>>>>>>>>>>>>>>>>>>>>>> this particular case but would it be an option > to > > > use > > > >>>>>>>>>>>>>>>> "hidden" > > > >>>>>>>>>>>>>>>>>>>>> attribute > > > >>>>>>>>>>>>>>>>>>>>>> in > > > >>>>>>>>>>>>>>>>>>>>>>> this case instead of CSS "display" ? > > > >>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>> 2) wicket-core.css instead of wicket-base.css ? > > > >>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>> On Mon, Jan 13, 2020 at 9:43 PM < > > > >> papega...@apache.org > > > >>>>>>>>>>>>>>>> wrote: > > > >>>>>>>>>>>>>>>>>>>>>>>> This is an automated email from the ASF > > > dual-hosted > > > >>>>> git > > > >>>>>>>>>>>>>>>>>>> repository. > > > >>>>>>>>>>>>>>>>>>>>>>>> papegaaij pushed a commit to branch csp > > > >>>>>>>>>>>>>>>>>>>>>>>> in repository > > > >>>>>>>>>>>>>>>> https://gitbox.apache.org/repos/asf/wicket.git > > > >>>>>>>>>>>>>>>>>>>>>>>> The following commit(s) were added to > > > refs/heads/csp > > > >>>>> by > > > >>>>>>>>>>>>>>>> this > > > >>>>>>>>>>>>>>>>>> push: > > > >>>>>>>>>>>>>>>>>>>>>>>> new 6d91a6a WICKET-6725: replace > > > >>>>> display:none > > > >>>>>> by > > > >>>>>>>>>>>>>>>>>>>>> wicket--hidden > > > >>>>>>>>>>>>>>>>>>>>>> css > > > >>>>>>>>>>>>>>>>>>>>>>>> class > > > >>>>>>>>>>>>>>>>>>>>>>>> 6d91a6a is described below > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> commit > 6d91a6a9e5c1d955a53571f9fb0f76262ac5c5d2 > > > >>>>>>>>>>>>>>>>>>>>>>>> Author: Emond Papegaaij < > > > emond.papega...@topicus.nl > > > >>>>>>>>>>>>>>>>>>>>>>>> AuthorDate: Mon Jan 13 20:43:01 2020 +0100 > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> WICKET-6725: replace display:none by > > > >>>>>>> wicket--hidden > > > >>>>>>>>>>>> css > > > >>>>>>>>>>>>>>>>>> class > > > >>>>>>>>>>>>>>>>>>>>>>>> --- > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>> .../src/main/java/org/apache/wicket/Component.java > > > >>>>>>> | 2 > > > >>>>>>>>>>>> +- > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > .../src/main/java/org/apache/wicket/Page.java > > > >>>>>> | > > > >>>>>>> 11 > > > >>>>>>>>>>>>>>>> +++++++ > > > >>>>> .../wicket/css/WicketBaseCSSResourceReference.java > > > >>>>>> | > > > >>>>>>> 36 > > > >>>>>>>>>>>>>>>>>>>>>>>> ++++++++++++++++++++++ > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > .../java/org/apache/wicket/css/wicket-base.css > > > >>>>>>> | 3 > > > >>>>>>>>>>>> ++ > > > >> .../apache/wicket/settings/ResourceSettings.java > > > >>>>>> | > > > >>>>>>> 31 > > > >>>>>>>>>>>>>>>>>>>>>>>> +++++++++++++++++++ > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > .../ajax/markup/html/AjaxIndicatorAppender.java > > > >>>>>>> | 2 > > > >>>>>>>>>>>> +- > > > >>>>>>>>>>>>>>>>>>>>>>>> 6 files changed, 83 insertions(+), 2 > > > >> deletions(-) > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>> a/wicket-core/src/main/java/org/apache/wicket/Component.java > > > >>>>>>>>>>>>>>>> > > > b/wicket-core/src/main/java/org/apache/wicket/Component.java > > > >>>>>>>>>>>>>>>>>>>>>>>> index 9da8ec5..d2da23b 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- > > > >>>>>>>>>>>>>>>> > > > a/wicket-core/src/main/java/org/apache/wicket/Component.java > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >>>>>>>>>>>>>>>> > > > b/wicket-core/src/main/java/org/apache/wicket/Component.java > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -2367,7 +2367,7 @@ public abstract class > > > >> Component > > > >>>>>>>>>>>>>>>>>>>>>>>> response.write(name); > > > >>>>>>>>>>>>>>>>>>>>>>>> response.write(" id=\""); > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>> response.write(getAjaxRegionMarkupId()); > > > >>>>>>>>>>>>>>>>>>>>>>>> - response.write("\" > > > >>>>>> style=\"display:none\" > > > >>>>>>>>>>>>>>>>>>>>>>>> data-wicket-placeholder=\"\"></"); > > > >>>>>>>>>>>>>>>>>>>>>>>> + response.write("\" > > > >>>>>>>>>>>> class=\"wicket--hidden\" > > > >>>>>>>>>>>>>>>>>>>>>>>> data-wicket-placeholder=\"\"></"); > > > >>>>>>>>>>>>>>>>>>>>>>>> response.write(name); > > > >>>>>>>>>>>>>>>>>>>>>>>> response.write(">"); > > > >>>>>>>>>>>>>>>>>>>>>>>> } > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>>>>>>>>>>>>>> > a/wicket-core/src/main/java/org/apache/wicket/Page.java > > > >>>>> b/wicket-core/src/main/java/org/apache/wicket/Page.java > > > >>>>>>>>>>>>>>>>>>>>>>>> index 3f0f5b5..3d70ad8 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- > > > >>>>>>>>>>>> a/wicket-core/src/main/java/org/apache/wicket/Page.java > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >>>>>>>>>>>> b/wicket-core/src/main/java/org/apache/wicket/Page.java > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -24,10 +24,13 @@ import java.util.Set; > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>> org.apache.wicket.authorization.UnauthorizedActionException; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>> org.apache.wicket.core.util.lang.WicketObjects; > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > >>>>>>>>>>>>>>>> org.apache.wicket.css.WicketBaseCSSResourceReference; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >> org.apache.wicket.feedback.FeedbackDelay; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >> org.apache.wicket.markup.MarkupException; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > org.apache.wicket.markup.MarkupStream; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > org.apache.wicket.markup.MarkupType; > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > org.apache.wicket.markup.head.CssHeaderItem; > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > >>>>> org.apache.wicket.markup.head.IHeaderResponse; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > org.apache.wicket.markup.html.WebPage; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>> org.apache.wicket.markup.resolver.IComponentResolver; > > > >>>>>>>>>>>>>>>>>>>>>>>> import org.apache.wicket.model.IModel; > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -1003,6 +1006,14 @@ public abstract class > Page > > > >>>>>> extends > > > >>>>>>>>>>>>>>>>>>>>>> MarkupContainer > > > >>>>>>>>>>>>>>>>>>>>>>>> } > > > >>>>>>>>>>>>>>>>>>>>>>>> } > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> + @Override > > > >>>>>>>>>>>>>>>>>>>>>>>> + public void renderHead(IHeaderResponse > > > >>>>>> response) > > > >>>>>>>>>>>>>>>>>>>>>>>> + { > > > >>>>>>>>>>>>>>>>>>>>>>>> + super.renderHead(response); > > > >>>>>>>>>>>>>>>>>>>>>>>> + response.render( > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > CssHeaderItem.forReference(getApplication().getResourceSettings().getWicketBaseCSS())); > > > >>>>>>>>>>>>>>>>>>>>>>>> + } > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> /** > > > >>>>>>>>>>>>>>>>>>>>>>>> * THIS METHOD IS NOT PART OF THE > > > WICKET > > > >>>>>>> PUBLIC > > > >>>>>>>>>>>>>>>> API. DO > > > >>>>>>>>>>>>>>>>>>> NOT > > > >>>>>>>>>>>>>>>>>>>>>> CALL. > > > >>>>>>>>>>>>>>>>>>>>>>>> * > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > a/wicket-core/src/main/java/org/apache/wicket/css/WicketBaseCSSResourceReference.java > > > >> > > > > b/wicket-core/src/main/java/org/apache/wicket/css/WicketBaseCSSResourceReference.java > > > >>>>>>>>>>>>>>>>>>>>>>>> new file mode 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> index 0000000..9247216 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- /dev/null > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > b/wicket-core/src/main/java/org/apache/wicket/css/WicketBaseCSSResourceReference.java > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -0,0 +1,36 @@ > > > >>>>>>>>>>>>>>>>>>>>>>>> +/* > > > >>>>>>>>>>>>>>>>>>>>>>>> + * Licensed to the Apache Software Foundation > > > (ASF) > > > >>>>>>>>>>>> under > > > >>>>>>>>>>>>>>>> one > > > >>>>>>>>>>>>>>>>>> or > > > >>>>>>>>>>>>>>>>>>>>> more > > > >>>>>>>>>>>>>>>>>>>>>>>> + * contributor license agreements. See the > > > NOTICE > > > >>>>>> file > > > >>>>>>>>>>>>>>>>>>> distributed > > > >>>>>>>>>>>>>>>>>>>>>> with > > > >>>>>>>>>>>>>>>>>>>>>>>> + * this work for additional information > regarding > > > >>>>>>>>>>>>>>>> copyright > > > >>>>>>>>>>>>>>>>>>>>> ownership. > > > >>>>>>>>>>>>>>>>>>>>>>>> + * The ASF licenses this file to You under > the > > > >>>>> Apache > > > >>>>>>>>>>>>>>>> License, > > > >>>>>>>>>>>>>>>>>>>>> Version > > > >>>>>>>>>>>>>>>>>>>>>>> 2.0 > > > >>>>>>>>>>>>>>>>>>>>>>>> + * (the "License"); you may not use this file > > > >> except > > > >>>>>> in > > > >>>>>>>>>>>>>>>>>>> compliance > > > >>>>>>>>>>>>>>>>>>>>>> with > > > >>>>>>>>>>>>>>>>>>>>>>>> + * the License. You may obtain a copy of the > > > >>>>> License > > > >>>>>> at > > > >>>>>>>>>>>>>>>>>>>>>>>> + * > > > >>>>>>>>>>>>>>>>>>>>>>>> + * > > > http://www.apache.org/licenses/LICENSE-2.0 > > > >>>>>>>>>>>>>>>>>>>>>>>> + * > > > >>>>>>>>>>>>>>>>>>>>>>>> + * Unless required by applicable law or > agreed to > > > >> in > > > >>>>>>>>>>>>>>>> writing, > > > >>>>>>>>>>>>>>>>>>>>> software > > > >>>>>>>>>>>>>>>>>>>>>>>> + * distributed under the License is > distributed > > > on > > > >>>>> an > > > >>>>>>>>>>>> "AS > > > >>>>>>>>>>>>>>>> IS" > > > >>>>>>>>>>>>>>>>>>>>> BASIS, > > > >>>>>>>>>>>>>>>>>>>>>>>> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY > KIND, > > > >>>>>> either > > > >>>>>>>>>>>>>>>> express > > > >>>>>>>>>>>>>>>>>>> or > > > >>>>>>>>>>>>>>>>>>>>>>>> implied. > > > >>>>>>>>>>>>>>>>>>>>>>>> + * See the License for the specific language > > > >>>>> governing > > > >>>>>>>>>>>>>>>>>>> permissions > > > >>>>>>>>>>>>>>>>>>>>> and > > > >>>>>>>>>>>>>>>>>>>>>>>> + * limitations under the License. > > > >>>>>>>>>>>>>>>>>>>>>>>> + */ > > > >>>>>>>>>>>>>>>>>>>>>>>> +package org.apache.wicket.css; > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > >>>>>>>>>>>>>>>> > org.apache.wicket.request.resource.CssResourceReference; > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> +public final class > WicketBaseCSSResourceReference > > > >>>>>>>>>>>> extends > > > >>>>>>>>>>>>>>>>>>>>>>>> CssResourceReference > > > >>>>>>>>>>>>>>>>>>>>>>>> +{ > > > >>>>>>>>>>>>>>>>>>>>>>>> + private static final long > > > serialVersionUID = > > > >>>>>>>>>>>>>>>>>>>>>>> 6795863553105608280L; > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + private static final > > > >>>>>>>>>>>> WicketBaseCSSResourceReference > > > >>>>>>>>>>>>>>>>>>>>> INSTANCE = > > > >>>>>>>>>>>>>>>>>>>>>>> new > > > >>>>>>>>>>>>>>>>>>>>>>>> WicketBaseCSSResourceReference(); > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + public static > > > WicketBaseCSSResourceReference > > > >>>>>>>>>>>> get() > > > >>>>>>>>>>>>>>>>>>>>>>>> + { > > > >>>>>>>>>>>>>>>>>>>>>>>> + return INSTANCE; > > > >>>>>>>>>>>>>>>>>>>>>>>> + } > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + private > WicketBaseCSSResourceReference() > > > >>>>>>>>>>>>>>>>>>>>>>>> + { > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>> super(WicketBaseCSSResourceReference.class, > > > >>>>>>>>>>>>>>>>>>>>>>>> "wicket-base.css"); > > > >>>>>>>>>>>>>>>>>>>>>>>> + } > > > >>>>>>>>>>>>>>>>>>>>>>>> +} > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> a/wicket-core/src/main/java/org/apache/wicket/css/wicket-base.css > > > >> b/wicket-core/src/main/java/org/apache/wicket/css/wicket-base.css > > > >>>>>>>>>>>>>>>>>>>>>>>> new file mode 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> index 0000000..9bbdd63 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- /dev/null > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >> b/wicket-core/src/main/java/org/apache/wicket/css/wicket-base.css > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -0,0 +1,3 @@ > > > >>>>>>>>>>>>>>>>>>>>>>>> +.wicket--hidden { > > > >>>>>>>>>>>>>>>>>>>>>>>> + display: none!important; > > > >>>>>>>>>>>>>>>>>>>>>>>> +} > > > >>>>>>>>>>>>>>>>>>>>>>>> \ No newline at end of file > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > a/wicket-core/src/main/java/org/apache/wicket/settings/ResourceSettings.java > > > >> > > > > b/wicket-core/src/main/java/org/apache/wicket/settings/ResourceSettings.java > > > >>>>>>>>>>>>>>>>>>>>>>>> index b76fbcd..68317f4 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > a/wicket-core/src/main/java/org/apache/wicket/settings/ResourceSettings.java > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > b/wicket-core/src/main/java/org/apache/wicket/settings/ResourceSettings.java > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -29,12 +29,14 @@ import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>> > org.apache.wicket.core.util.resource.locator.IResourceStreamLocator; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>> > org.apache.wicket.core.util.resource.locator.ResourceStreamLocator; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > org.apache.wicket.core.util.resource.locator.caching.CachingResourceStreamLocator; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > org.apache.wicket.css.ICssCompressor; > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > >>>>>>>>>>>>>>>> org.apache.wicket.css.WicketBaseCSSResourceReference; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>> org.apache.wicket.javascript.IJavaScriptCompressor; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>> org.apache.wicket.markup.head.PriorityFirstComparator; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>> > org.apache.wicket.markup.head.ResourceAggregator.RecordedHeaderItem; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>> org.apache.wicket.markup.html.IPackageResourceGuard; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>> > > > org.apache.wicket.markup.html.SecurePackageResourceGuard; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>> org.apache.wicket.request.http.WebResponse; > > > >>>>>>>>>>>>>>>>>>>>>>>> +import > > > >>>>>>>>>>>>>>>> > org.apache.wicket.request.resource.CssResourceReference; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > org.apache.wicket.request.resource.caching.FilenameWithVersionResourceCachingStrategy; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>> > org.apache.wicket.request.resource.caching.IResourceCachingStrategy; > > > >>>>>>>>>>>>>>>>>>>>>>>> import > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > org.apache.wicket.request.resource.caching.NoOpResourceCachingStrategy; > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -172,6 +174,8 @@ public class > ResourceSettings > > > >>>>>>>>>>>>>>>> implements > > > >>>>>>>>>>>>>>>>>>>>>>>> IPropertiesFactoryContext > > > >>>>>>>>>>>>>>>>>>>>>>>> false); > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> private boolean encodeJSessionId > = > > > >> false; > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + private CssResourceReference > > > wicketBaseCSS = > > > >>>>>>>>>>>>>>>>>>>>>>>> WicketBaseCSSResourceReference.get(); > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> /** > > > >>>>>>>>>>>>>>>>>>>>>>>> * Configures Wicket's default > > > >>>>>>>>>>>> ResourceLoaders.<br> > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -770,4 +774,31 @@ public class > ResourceSettings > > > >>>>>>>>>>>>>>>> implements > > > >>>>>>>>>>>>>>>>>>>>>>>> IPropertiesFactoryContext > > > >>>>>>>>>>>>>>>>>>>>>>>> this.encodeJSessionId = > > > >>>>>>>>>>>> encodeJSessionId; > > > >>>>>>>>>>>>>>>>>>>>>>>> return this; > > > >>>>>>>>>>>>>>>>>>>>>>>> } > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + /** > > > >>>>>>>>>>>>>>>>>>>>>>>> + * Returns the resource reference of > the > > > >> base > > > >>>>>>>>>>>>>>>> stylesheet > > > >>>>>>>>>>>>>>>>>>> for > > > >>>>>>>>>>>>>>>>>>>>>>>> Wicket. This stylesheet contains > > > >>>>>>>>>>>>>>>>>>>>>>>> + * some lowlevel styling used by > Wicket. > > > >>>>>>>>>>>>>>>>>>>>>>>> + * > > > >>>>>>>>>>>>>>>>>>>>>>>> + * @return The resource reference of > the > > > >> base > > > >>>>>>>>>>>>>>>> stylesheet > > > >>>>>>>>>>>>>>>>>>> for > > > >>>>>>>>>>>>>>>>>>>>>>>> Wicket. > > > >>>>>>>>>>>>>>>>>>>>>>>> + */ > > > >>>>>>>>>>>>>>>>>>>>>>>> + public CssResourceReference > > > >>>>> getWicketBaseCSS() > > > >>>>>>>>>>>>>>>>>>>>>>>> + { > > > >>>>>>>>>>>>>>>>>>>>>>>> + return wicketBaseCSS; > > > >>>>>>>>>>>>>>>>>>>>>>>> + } > > > >>>>>>>>>>>>>>>>>>>>>>>> + > > > >>>>>>>>>>>>>>>>>>>>>>>> + /** > > > >>>>>>>>>>>>>>>>>>>>>>>> + * Replaces the base stylesheet for > > > Wicket. > > > >>>>>>>>>>>>>>>> Changes made > > > >>>>>>>>>>>>>>>>>>> to > > > >>>>>>>>>>>>>>>>>>>>> the > > > >>>>>>>>>>>>>>>>>>>>>>>> styling can break functionality > > > >>>>>>>>>>>>>>>>>>>>>>>> + * like {@link > > > >>>>>>>>>>>>>>>>>>>>>> > Component#setOutputMarkupPlaceholderTag(boolean)}, > > > >>>>>>>>>>>>>>>>>>>>>>>> causing components that should > > > >>>>>>>>>>>>>>>>>>>>>>>> + * not be visible to be displayed. > Make > > > sure > > > >>>>>> the > > > >>>>>>>>>>>>>>>>>>> replacement > > > >>>>>>>>>>>>>>>>>>>>>>>> stylesheet has matching definitions > > > >>>>>>>>>>>>>>>>>>>>>>>> + * for the corresponding sections in > the > > > >>>>> Wicket > > > >>>>>>>>>>>>>>>> version. > > > >>>>>>>>>>>>>>>>>>>>>>>> + * > > > >>>>>>>>>>>>>>>>>>>>>>>> + * @param wicketBaseCSS > > > >>>>>>>>>>>>>>>>>>>>>>>> + * The replacement > styleheet. > > > >>>>>>>>>>>>>>>>>>>>>>>> + * @return {@code this} object for > > > chaining > > > >>>>>>>>>>>>>>>>>>>>>>>> + */ > > > >>>>>>>>>>>>>>>>>>>>>>>> + public ResourceSettings > > > >>>>>>>>>>>>>>>>>>>>> setWicketBaseCSS(CssResourceReference > > > >>>>>>>>>>>>>>>>>>>>>>>> wicketBaseCSS) > > > >>>>>>>>>>>>>>>>>>>>>>>> + { > > > >>>>>>>>>>>>>>>>>>>>>>>> + this.wicketBaseCSS = > > > wicketBaseCSS; > > > >>>>>>>>>>>>>>>>>>>>>>>> + return this; > > > >>>>>>>>>>>>>>>>>>>>>>>> + } > > > >>>>>>>>>>>>>>>>>>>>>>>> } > > > >>>>>>>>>>>>>>>>>>>>>>>> diff --git > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/AjaxIndicatorAppender.java > > > >> > > > > b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/AjaxIndicatorAppender.java > > > >>>>>>>>>>>>>>>>>>>>>>>> index d0fee97..ab1c0a1 100644 > > > >>>>>>>>>>>>>>>>>>>>>>>> --- > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/AjaxIndicatorAppender.java > > > >>>>>>>>>>>>>>>>>>>>>>>> +++ > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >> > > > > b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/AjaxIndicatorAppender.java > > > >>>>>>>>>>>>>>>>>>>>>>>> @@ -95,7 +95,7 @@ public class > > > AjaxIndicatorAppender > > > >>>>>>>>>>>>>>>> extends > > > >>>>>>>>>>>>>>>>>>>>> Behavior > > > >>>>>>>>>>>>>>>>>>>>>>>> > super.afterRender(component); > > > >>>>>>>>>>>>>>>>>>>>>>>> final Response r = > > > >>>>>>>>>>>> component.getResponse(); > > > >>>>>>>>>>>>>>>>>>>>>>>> - r.write("<span > > > >>>>> style=\"display:none;\" > > > >>>>>>>>>>>>>>>>>> class=\""); > > > >>>>>>>>>>>>>>>>>>>>>>>> + r.write("<span > > > >>>>> class=\"wicket--hidden\" > > > >>>>>>>>>>>>>>>>>>> class=\""); > > > >>>>>>>>>>>>>>>>>>>>>>>> r.write(getSpanClass()); > > > >>>>>>>>>>>>>>>>>>>>>>>> r.write("\" "); > > > >>>>>>>>>>>>>>>>>>>>>>>> r.write("id=\""); > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>>>> -- > > > >>>>>>>>>>>>>>>>>>>>>> WBR > > > >>>>>>>>>>>>>>>>>>>>>> Maxim aka solomax > > > >>>>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>>> -- > > > >>>>>>>>>>>>>>>>>>>> WBR > > > >>>>>>>>>>>>>>>>>>>> Maxim aka solomax > > > >>>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>>>> -- > > > >>>>>>>>>>>>>>>>>>> WBR > > > >>>>>>>>>>>>>>>>>>> Maxim aka solomax > > > >>>>>>>>>>>>>>>>>>> > > > >>>>>>>>>>>>>>>>> -- > > > >>>>>>>>>>>>>>>>> WBR > > > >>>>>>>>>>>>>>>>> Maxim aka solomax > > > >>>>>>>>>>>>>>> -- > > > >>>>>>>>>>>>>>> WBR > > > >>>>>>>>>>>>>>> Maxim aka solomax > > > >>>>>>>>>>>>>>> > > > >>>>>>>>>>>> -- > > > >>>>>>>>>>>> WBR > > > >>>>>>>>>>>> Maxim aka solomax > > > >>>>> -- > > > >>>>> WBR > > > >>>>> Maxim aka solomax > > > >>>>> > > > > >