Code review comment for lp:~jeremywootten/pantheon-terminal/fix-1208140

Revision history for this message
Jeremy Wootten (jeremywootten) wrote :

Thanks for the advice, I will do that.

On 27 August 2013 15:26, Sergey "Shnatsel" Davidoff <email address hidden>wrote:

> Review: Needs Fixing
>
> It's better to make a new .desktop file called "Open Terminal Here" with
> "NoDisplay=True" key instead of changing the existing .desktop file. This
> will make "Terminal" show up in app launchers (e.g. Slingshot) and "Open
> Terminal Here" in Open With menus.
>
> So I'm adding my name to the long list of "Needs Fixing" reviewers :D
> But thanks for tackling this issue!
> --
>
> https://code.launchpad.net/~jeremywootten/pantheon-terminal/fix-1208140/+merge/179597
> You are the owner of lp:~jeremywootten/pantheon-terminal/fix-1208140.
>

--
Jeremy Wootten
GPG Key ID CB585BCD
Key Fingerprint 37C0 3C2A A6D4 E45B BA7C 4328 2DF2 1882 CB58 5BCD

« Back to merge proposal