<html>
    <head>
      <base href="https://bugzilla.gnome.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Epiphany displays incorrect name in gnome-shell app menu"
   href="https://bugzilla.gnome.org/show_bug.cgi?id=752258#c26">Comment # 26</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Epiphany displays incorrect name in gnome-shell app menu"
   href="https://bugzilla.gnome.org/show_bug.cgi?id=752258">bug 752258</a>
              from <span class="vcard"><a href="page.cgi?id=describeuser.html&login=jadahl%40gmail.com" title="Jonas Ådahl <jadahl@gmail.com>"> <span class="fn">Jonas Ådahl</span></a>
</span></b>
        <pre>(In reply to Florian Müllner from <a href="show_bug.cgi?id=752258#c25">comment #25</a>)
<span class="quote">> (In reply to Jonas Ådahl from <a href="show_bug.cgi?id=752258#c24">comment #24</a>)
> > The question is rather, how do we get a valid name to pass to
> > xdg_surface.set_app_id, which id should be the base name of the .desktop
> > file. Can we force GtkApplication apps to set the prgname to this?

> To the basename of the .desktop file? No, we don't know that. GtkApplication
> does set a default prgname if the app didn't set it itself[0] (which happens
> to match the pre-dbus-activation convention for .desktop file names), and
> apps with a different naming scheme (hopefully) call it themselves to hint
> at the right name. So using g_get_prgname() for set_app_id() should be a
> good pick if the goal is to associate the app with its .desktop file, or
> just go back to the previously used gdk_get_program_class() - that's what's
> used on X, and some fringe app might call gdk_set_program_class() directly
> instead of g_set_prgname()
> </span >

Won't this mean it'll be incorrect for D-Bus activatable applications with no
possibility to fix it? Or do you mean D-Bus activatable applications should set
the prgname to the interface name?

gdk_get_program_class() will most likely never be correct. Prior to the patch,
gedit would be "Gedit" and gnome-terminal would be "Gnome-terminal" IIRC.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>