<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    -----BEGIN PGP SIGNED MESSAGE-----<br>
    Hash: SHA1<br>
    <br>
    thanks I add the line (this->Internal->addProxy(group, name,
    showInMenu->GetAttribute("icon"));) in my paraview sources and
    the icons are visible again. thanks.<br>
    <br>
    But what about the toolbar?. the "show_in_toolbar" option is also
    lost.<br>
    <br>
    adding these lines(after the line 806 in your patch) make the
    show_in_toolbar work again.<br>
    <br>
              int show_in_toolbar = 0;<br>
              hint->GetScalarAttribute("show_in_toolbar",
    &show_in_toolbar);<br>
              category.ShowInToolbar = category.ShowInToolbar ||
    (show_in_toolbar == 1);<br>
    <br>
    in this case we have to put a new tag in the xml hints.<br>
    <br>
           <ShowInMenu category="PXDMF"
    icon=":/PXDMFIcons/AnnotateFieldData.png" show_in_toolbar="1" /><br>
    <br>
    <br>
    I don't know if this happens only in my 4.3-RC1 but the
    View->Toolbars menu show  2 "VCR Controls" submenus. Can anyone
    confirm this.<br>
    <br>
    thanks<br>
    Felipe<br>
    <br>
    <br>
    <br>
    <br>
    Le 08/01/2015 15:48, Utkarsh Ayachit a Ã©crit :<br>
    <span style="white-space: pre;">> Here's the change:<br>
      >    <a class="moz-txt-link-freetext" href="http://review.source.kitware.com/#/c/18740/">http://review.source.kitware.com/#/c/18740/</a><br>
      ><br>
      > Should make it into the official release of 4.3.<br>
      ><br>
      > Utkarsh<br>
      ><br>
      > On Thu, Jan 8, 2015 at 9:31 AM, Utkarsh Ayachit<br>
      > <a class="moz-txt-link-rfc2396E" href="mailto:utkarsh.ayachit@kitware.com"><utkarsh.ayachit@kitware.com></a> wrote:<br>
      >> Good catch. I don't exactly know what the PipelineIcon
      hint is.<br>
      >> Looking at the code I don't think it was intended to be
      functional.<br>
      >> I'll track down the developer and figure that part out. I
      am adding<br>
      >> support to add a new attribute "icon" to the
      <ShowInMenu > element.<br>
      >><br>
      >> e.g.<br>
      >>    <Hints><br>
      >>       <ShowInMenu icon="...." /><br>
      >>   </Hints><br>
      >><br>
      >> Utkarsh<br>
      >><br>
      >> On Thu, Jan 8, 2015 at 7:52 AM, Felipe Bordeu<br>
      >> <a class="moz-txt-link-rfc2396E" href="mailto:felipe.bordeu@ec-nantes.fr"><felipe.bordeu@ec-nantes.fr></a> wrote:<br>
      >>><br>
      > Hello,<br>
      ><br>
      > I'm porting my plugin into version 4.3 and I'm unable to set
      the icons<br>
      > for my filters.<br>
      > I tried :
      (<a class="moz-txt-link-freetext" href="http://www.paraview.org/Wiki/ParaView/Plugin_HowTo(">http://www.paraview.org/Wiki/ParaView/Plugin_HowTo(</a><br>
      ><br>
      > ...<br>
      > Â  <Hints><br>
      > Â Â Â  <ShowInMenu category="PXDMF" /><br>
      > Â Â Â  <PipelineIcon
      name=":/PXDMFIcons/AnnotateFieldData.png"  /><br>
      > Â  </Hints><br>
      > </SourceProxy><br>
      ><br>
      > ...<br>
      > but does not work. Searching for the keyword "PipelineIcon"
      in the<br>
      > Paraview sources gives no results.<br>
      ><br>
      > also :  the  ShowInMenu shows the filter in a new submenu
      but the is no<br>
      > new toolbar as in the 4.2.<br>
      ><br>
      > in the 4.2 version this was my GUI xml<br>
      ><br>
      > <ParaViewFilters><br>
      > Â  <Category name="PXDMF" menu_label="&amp;PXDMF"
      show_in_toolbar="1"><br>
      > Â Â Â Â Â  <!-- adds a new category and then adds our
      filter to it --><br>
      > Â Â Â Â Â  <Filter name="AnnotateFieldData"<br>
      > icon=":/PXDMFIcons/AnnotateFieldData.png" /><br>
      > Â  </Category><br>
      > </ParaViewFilters><br>
      ><br>
      > any ideas.<br>
      ><br>
      > Thanks to all. And thanks for this new version.<br>
      > Felipe<br>
      ><br>
      >>><br>
      >>><br>
      >>> _______________________________________________<br>
      >>> Paraview-developers mailing list<br>
      >>> <a class="moz-txt-link-abbreviated" href="mailto:Paraview-developers@paraview.org">Paraview-developers@paraview.org</a><br>
      >>>
      <a class="moz-txt-link-freetext" href="http://public.kitware.com/mailman/listinfo/paraview-developers">http://public.kitware.com/mailman/listinfo/paraview-developers</a></span><br>
    <br>
    - -- <br>
    Felipe Bordeu Weldt<br>
    Ingénieur de Recherche<br>
    - -------------------------------------<br>
    Tél. : 33 (0)2 40 37 16 57<br>
    Fax. : 33 (0)2 40 74 74 06<br>
    <a class="moz-txt-link-abbreviated" href="mailto:Felipe.Bordeu@ec-nantes.fr">Felipe.Bordeu@ec-nantes.fr</a><br>
    Institut GeM - UMR CNRS 6183<br>
    Ã‰cole Centrale Nantes<br>
    1 Rue de La Noë, 44321 Nantes, FRANCE<br>
    - -------------------------------------<br>
    <br>
    -----BEGIN PGP SIGNATURE-----<br>
    Version: GnuPG v1<br>
    <br>
    iQEcBAEBAgAGBQJUr6iHAAoJEE/fMfNgU9/D/VUIAJKWqm+fIhqGckxda4bqrxZI<br>
    2ztfvVU8h9PbJN1LFpf8pSHTMETI4Lf6LN8YDuUH6F5YI/jEZHl7wF1N1ygaDHI4<br>
    6hllc1/lyJPd3n3LROoPNd9/A2eUv9oNZzW1u7LjHbKdq/M1yi1XKYUzFKJdgAlb<br>
    4W060OBKlalGcgasHxWa7+U5breML/3D11Why+HP/oTAA4p9HdRIyUxjpBjmYxRk<br>
    luS4bnEi9RlD1vDFzldYQm2fzNHJ4jllq35QbYcTzkOHSTNtRzwAqXGHc1J0z0V7<br>
    Wi9bUu1uKKfYC0pxHYrjYh1/PlJ5+cVoFW9+cPIOUwB0/Ex95Tr2TzyiO62ZLPI=<br>
    =ta6C<br>
    -----END PGP SIGNATURE-----<br>
    <br>
  </body>
</html>