Skip to content
This repository has been archived by the owner on Jul 23, 2022. It is now read-only.

Added GenericName for app on Linux #439

Merged
merged 1 commit into from
Apr 26, 2020
Merged

Added GenericName for app on Linux #439

merged 1 commit into from
Apr 26, 2020

Conversation

SaeedBaig
Copy link
Contributor

Added "GenericName" field for Bitwarden's desktop option for Linux, so that its icon displays a useful subtitle in menus and taskbars (demonstrated in screenshots below).

Since the convention is a short descriptive label of what the app is, and other password managers like KeePassXC and Keeper use "Password Manager", I also used "Password Manager".

Pic 1. Example of what other apps look like In the KDE taskbar:
Pic1

Pic 2. What Bitwarden looks like before change:
Pic2

Pic 3. What Bitwarden looks like after change:
Pic3

Pic 4. Bitwarden in KDE Application Launcher before change:
Pic4

Pic 5. Bitwarden in KDE Application Launcher after change:
Pic5

This brings its appearance/behaviour in line with other apps on Linux.

There shouldn't be any other unexpected changes to the app on Linux, since the generated Desktop Entry file is the same except the new "GenericName".

Pic 6. Desktop entry file before change:
Pic6

Pic 7. Desktop entry file after change:
Pic7

(built & tested on Kubuntu 18.04)

Added desktop option for building the app for Linux, just so "GenericName" could be specified so it looks better on Linux ("Name" and "Type" there because they're required).
@CLAassistant
Copy link

CLAassistant commented Apr 26, 2020

CLA assistant check
All committers have signed the CLA.

@kspearrin kspearrin merged commit 4fd3eec into bitwarden:master Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants