<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Buddy,<br>
    <br>
    I have a couple of comments on this:<br>
    <br>
    1. the default behaviour should remain the same as before applying
    this patch which is to be enabled without a button.&nbsp; To me it looks
    like you are changing this widget to always require a button to
    enable the tooltips, which will break a lot of existing applications
    that don't have a spot for the button.&nbsp;&nbsp; You need to make the button
    an optional behaviour.<br>
    <br>
    2. calculating this.mainPath: we already have Fusion.fusionURL which
    holds a pointer to where fusion.js was loaded from and your images
    should be relative to that or relative to the template directory.&nbsp;
    The way you are getting that image URL just looks odd to me and not
    the usual Fusion way of doing things.<br>
    <br>
    Cheers,<br>
    <br>
    Mike<br>
    <br>
    <br>
    On 29/07/2011 5:35 AM, Buddy Hu wrote:
    <blockquote
cite="mid:71474561E5BE0848A79F662BC462D4D203E731@005-TK5MPN1-012.MGDADSK.autodesk.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      <meta name="Generator" content="Microsoft Word 14 (filtered
        medium)">
      <style><!--
/* Font Definitions */
@font-face
        {font-family:SimSun;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:SimSun;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"\@SimSun";
        panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-compose;
        font-family:"Calibri","sans-serif";
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
      <div class="WordSection1">
        <p class="MsoNormal">Hi List,<o:p></o:p></p>
        <p class="MsoNormal"><o:p>&nbsp;</o:p></p>
        <p class="MsoNormal">Please help to review the patch: <a
            moz-do-not-send="true"
            href="https://trac.osgeo.org/fusion/ticket/471">
            https://trac.osgeo.org/fusion/ticket/471</a><o:p></o:p></p>
        <p class="MsoNormal"><o:p>&nbsp;</o:p></p>
        <p class="MsoNormal">This patch intends to enhance the &#8220;MapTip&#8221;
          widget, makes this widget to be a UI widget(a button), so that
          the users can press it to enable/disable the map tooltips.<o:p></o:p></p>
        <p class="MsoNormal">The motivation of this enhancement is to
          provide an opportunity for the users to reduce the overload of
          Server by disable the map tooltips.<o:p></o:p></p>
        <p class="MsoNormal"><o:p>&nbsp;</o:p></p>
        <p class="MsoNormal">Thanks in advance!<o:p></o:p></p>
        <p class="MsoNormal">Buddy<o:p></o:p></p>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
fusion-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:fusion-dev@lists.osgeo.org">fusion-dev@lists.osgeo.org</a>
<a class="moz-txt-link-freetext" href="http://lists.osgeo.org/mailman/listinfo/fusion-dev">http://lists.osgeo.org/mailman/listinfo/fusion-dev</a>
</pre>
    </blockquote>
  </body>
</html>