Page 3 of 4
Re: New component pe:statusBar
Posted: 14 Jun 2012, 10:53
by Oleg
Thanks. But it still generates JS script. Right? We need a lightweight without JS script to place it e.g. in DataTable column. Just an output, like all h:output*. Otherwise there are performance problems with big tables, like p:select* if you have 100+ such selects on a page.
And we need:
- Tooltip (title) displaying the same labelTemplate on mouseover. We show e.g. progress bars in table column (hit list after search) with title "Relevance {value}%".
- labelPosition ("left" | "top" | "right" | "bottom" | "middle" (default)).
Re: New component pe:statusBar
Posted: 14 Jun 2012, 11:18
by cagatay.civici
Added displayOnly attribute not to render script, label position is easy to customize with css and no action taken for tooltip stuff. Maybe for future for p:progressBar as it doesn't have high priority right now.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 11:36
by Oleg
Great, thanks for the fast response! Tooltip is missing
What is about to render "title" per default, similar what you already do:
Code: Select all
if(labelTemplate != null && value != 0) {
// render style
...
String label = labelTemplate.replaceAll("\\{value\\}", String.valueOf(value));
writer.write(label);
writer.writeAttribute("title", label, null); // this line is new
}
This is just one line of code. Could you add it please and save our life?
Thanks again.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 11:44
by cagatay.civici
Why do we need to display same text both on label and on tooltip? Shouldn't tooltip provide advisory information?
Re: New component pe:statusBar
Posted: 14 Jun 2012, 11:54
by Oleg
My mistake. My use case - no label, only tooltip. It makes sence because progress bar is too small within table column. Maybe a new attribute "onlyTooltip" (true | false)? If true - no label is shown, only tooltip.
If it's too time expensive for you, forget this post. Thanks.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 12:15
by Oleg
Like this
Re: New component pe:statusBar
Posted: 14 Jun 2012, 12:33
by cagatay.civici
I've taken a note about this, I think it would be useful for sure, will give an update today or tomorrow. Thanks for the feedback Oleg.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 12:40
by Oleg
Thank you too, Cagatay! We will remove started work on pe:outputProgressBar. No needs in this comp. Thanks again.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 12:50
by cagatay.civici
Yes, let's not work on things that overlap too much.
Re: New component pe:statusBar
Posted: 14 Jun 2012, 15:52
by Scott Warner
You guys are great. Thank you.