----- Original Message ----- > From: "Greg Sheremeta" <[email protected]> > To: "Vojtech Szocs" <[email protected]> > Cc: [email protected], "Alexander Wels" <[email protected]> > Sent: Thursday, June 5, 2014 3:44:14 PM > Subject: Re: jQuery? > > ----- Original Message ----- > > From: "Vojtech Szocs" <[email protected]> > > To: "Greg Sheremeta" <[email protected]> > > Cc: [email protected], "Alexander Wels" <[email protected]> > > Sent: Thursday, June 5, 2014 9:26:19 AM > > Subject: Re: jQuery? > > > > > > ----- Original Message ----- > > > From: "Greg Sheremeta" <[email protected]> > > > To: [email protected] > > > Cc: "Alexander Wels" <[email protected]>, "Vojtech Szocs" > > > <[email protected]> > > > Sent: Wednesday, June 4, 2014 3:19:23 AM > > > Subject: jQuery? > > > > > > Does anyone see an issue with adding jQuery to GwtHostPage? Some > > > PatternFly > > > widgets require bootstrap.min.js which requires jQuery. > > > > My understanding is that PatternFly is essentially a "skin" of Bootstrap > > CSS, > > which doesn't imply the need for using Bootstrap _JavaScript_ library > > (which > > needs jQuery) to deal with widgets in JavaScript runtime. > > > > Any JavaScript-enabled widgets (like tooltips) do require bootstrap.js and > jquery.js.
I'm not sure this is the only way. gwtbootstrap3 could do the same thing, after all its code becomes JavaScript in the end. I think there is no real difference (conceptually) between bootstrap.js vs gwtbootstrap3, each one can use JavaScript to implement widgets like tooltips. In the end it boils down to styling DOM elements with CSS classes, so Bootstrap CSS framework is the lowest common denominator. > The only reason I didn't add them in the main PatternFly patch > is that we didn't need them back then. But now we need them. Hm, I assume you mean this: http://getbootstrap.com/javascript/#tooltips these are jQuery plugins, apparently bootstrap.js builds on top of jQuery to ease the work with DOM elements. For example: $('#example').tooltip(options) will register mouse listeners on given element that control the tooltip. Anyway, the tooltip behavior is just JavaScript + setting CSS classes, not really related to jQuery. However, I found this: https://github.com/gwtbootstrap3/gwtbootstrap3/blob/master/gwtbootstrap3/src/main/java/org/gwtbootstrap3/client/ui/Tooltip.java so it seems that gwtbootstrap3 _does_ support things like tooltips. EDIT: see my comment below, Tooltip.java depends on jQuery, you're right. > > > I thought that the purpose of "gwtbootstrap3" GWT module (integration with > > Bootstrap CSS framework) was to provide such widgets in context of GWT app. > > If not, what is then the purpose of "gwtbootstrap3" module? > > gwtbootstrap3 provides renderings of GWT widgets that are compatible with > bootstrap css. It does not rewrite the jQuery code into GWT-JavaScript code. > > > > > In other words, doesn't "gwtbootstrap3" module provide us with all widgets > > we need? > > I wouldn't say "provide." It enables using the widgets more easily from GWT. > > Take a look at > https://github.com/gwtbootstrap3/gwtbootstrap3/blob/master/gwtbootstrap3/src/main/java/org/gwtbootstrap3/client/ui/Tooltip.java > line 521 on down. > > It simply wraps everything for ease-of-use from GWT / UIBinder. You are correct. So gwtbootstrap3 requires jQuery. I didn't know this, just realized this now after looking at Tooltip.java code. So gwtbootstrap3 is just a thin GWT wrapper to bootstrap.js which implies the need for jQuery. (Originally I thought it's not just a thin wrapper.) Considering above, I'm OK with adding jQuery to GWT UI host page. BTW, we could do this via <script> in GwtCommon.gwt.xml which will cause the script to be automatically injected into GWT compilation (permutation) output, so we wouldn't need to modify host page HTML. However, question is, how/where to maintain jQuery script file(s). > > Greg > > > > > > > > > Thanks, > > > Greg > > > > > > Greg Sheremeta > > > Red Hat, Inc. > > > Sr. Software Engineer, RHEV > > > Cell: 919-807-1086 > > > [email protected] > > > > > > > > > _______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
