| Summary: | Type column should not show 'Comment' from *.desktop files | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-kio | Reporter: | tguen <tguenther.dev> |
| Component: | general | Assignee: | David Faure <faure> |
| Status: | REPORTED --- | ||
| Severity: | normal | CC: | bugseforuns, count.negative, kde, kdelibs-bugs, nate, tbruno-kde, tguenther.dev |
| Priority: | NOR | Keywords: | junior-jobs |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
screenshot of desktop files
patch |
||
This is from KFileItem (I think) whose (mime)Comment also reads the Comment. Re-assigning there but I don't think this will be changed or "fixed" for Dolphin. Bug persists. Operating System: Arch Linux KDE Plasma Version: 5.15.3 KDE Frameworks Version: 5.56.0 Qt Version: 5.12.2 Yes it's still in KFileItem::mimeComment. There the comment field is read and returned when file type is application/x-desktop. But why was that implemented as first? Created attachment 124809 [details]
patch
since phabricator won't let me submit a patch, it's here
Let's fix your Phab access so you can submit a patch. Are you having trouble logging in, or submitting it once you're logged in? (In reply to Nate Graham from comment #5) > Let's fix your Phab access so you can submit a patch. Are you having trouble > logging in, or submitting it once you're logged in? meeh, was a me too stupid problem, and by the way: Happy new Year! Was this patch ever submitted? |
Created attachment 106609 [details] screenshot of desktop files If a *.desktop file contains a Comment field, it is used as the type by Dolphin. This causes strange 'types' to be displayed, such as "Browse the Web" for firefox.