Skip to content

feat(assets): add a .desktop entry#107

Closed
MingcongBai wants to merge 1 commit intomicrosoft:mainfrom
AOSC-Tracking:upstream/desktop-entry
Closed

feat(assets): add a .desktop entry#107
MingcongBai wants to merge 1 commit intomicrosoft:mainfrom
AOSC-Tracking:upstream/desktop-entry

Conversation

@MingcongBai
Copy link
Contributor

Add a .desktop entry for Edit. It expects edit.svg (also found in assets/) to be installed in one of the icon paths.

Conformance to FreeDesktop.org's Desktop Entry Specification with desktop-file-validate(1).

Add a .desktop entry for Edit. It expects edit.svg (also found in assets/)
to be installed in one of the icon paths.

Conformance to FreeDesktop.org's Desktop Entry Specification with
desktop-file-validate(1).

Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
Comment on lines +14 to +15
Keywords=text;txt;editor;note;notepad;
Keywords[zh_CN]=text;txt;editor;note;notepad;文字;文本;编辑器;笔记;记事本;wenzi;wenben;bianjiqi;biji;jishiben;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Helix only lists "text" and "editor" as keywords: https://github.com/helix-editor/helix/blob/3a6c9747b8de3350c38ba504916233d9d4c33c5c/contrib/Helix.desktop#L82
Should we do the same and only use this?

Keywords=text;editor

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we do the same and only use this?

This was done to align with KWrite, which in my opinion is quite easy to search for - being able to find it with "Windows-like" keywords is convenient.

Copy link

@driver1998 driver1998 May 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edit and Notepad are two very different things, even on Windows.
I don't think people expect searching for "记事本" (Notepad) and getting a TUI editor...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edit and Notepad are two very different things, even on Windows. I don't think people expect searching for "记事本" (Notepad) and getting a TUI editor...

Sure, but notepad.exe and wordpad.exe being different things are really beside the point - both KDE and GNOME relies heavily on search-based navigation, and so does more traditional desktop environments (more experienced users are more likely to just use Alt-F2), same goes for WMs.

The second point I can agree with, but then this .desktop entry is useless - "why wouldn't they just run edit in a terminal?"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And, to clarify, I do not expect everyone to install or package this .desktop entry, it's here as a reference.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've discussed this and would currently prefer just text;editor like Helix uses. Could you make this change? If we ever take on more complex features (like VS Code) we could extend the list.

@lhecker lhecker added the E-needs-discussion Needs discussion between maintainers. label May 21, 2025
@lhecker lhecker removed the E-needs-discussion Needs discussion between maintainers. label May 27, 2025
@DHowett DHowett closed this in #420 Jun 4, 2025
DHowett pushed a commit that referenced this pull request Jun 4, 2025
Closes #107

Co-authored-by: Mingcong Bai <jeffbai@aosc.io>
@DHowett
Copy link
Member

DHowett commented Jun 4, 2025

Thanks! We weren't able to push a quick fix to your pr, so we credited you in #420 :)

Lou32Verbose pushed a commit to Lou32Verbose/edit that referenced this pull request Jan 11, 2026
Closes microsoft#107

Co-authored-by: Mingcong Bai <jeffbai@aosc.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants