Add database name, color, and icon options for unlock view

[techninja1008]

Closes #10783

Adds three database configuration options (stored as public custom data) that allow a database to have a public name/summary, color, and/or icon to be displayed on the unlock screen. This information is configured in the Database Settings and stored in the database public custom data (ie, unencrypted).

The name/summary is stored in KPXC_PUBLIC_NAME, the color is stored in KPXC_PUBLIC_COLOR, and the icon is stored in KPXC_PUBLIC_ICON.

Screenshots

Testing strategy

I have extensively performed manual testing with different values of both the message (empty, short text, long text etc) and color (empty, varying colors, invalid colors etc). I've also tested opening/closing the database, starting keepassxc after it was the last DB opened etc to approach the unlock UI from different code paths.

Type of change

• ✅ New feature (change that adds functionality)

[techninja1008]

@droidmonkey let me know if there is anything else you need to make this mergeable.

[droidmonkey]

Seemed fine on first pass good work

[techninja1008]

I quite like both the second and third ones, however I still quite like the original banner style and how in your face it was

[droidmonkey]

The original was ok, but looked like our warning/error banner to be honest. I wouldn't personally use this feature with the original look, but with the 2nd or 3rd I would. I also wanted to separate the database name from the color since some people would want one or the other. I'll let @phoerious weigh in before I make any more changes.

[RJVB]

If the goal is to make it easier to distinguish DBs from the unlock screen, how about showing the icon of the root collection? IMHO that'll work better for that particular purpose than a description, and it's less in-your-face than the big (IMHO aggressive) colour patches shown above.

FWIW, on OSes that support this the main DB icon could even be put in the file metadata so it shows in the Finder/Explorer/WhatEver.

[droidmonkey]

That is a non-starter, it would expose internal data. However picking one of our standard icons as the "database icon" is not a bad idea if people don't want to use the color option.

[RJVB]

On Monday August 26 2024 08:29:38 Jonathan White wrote: That is a non-starter, it would expose internal data

exposing the icon? There must be a way to store it (or an additional one) without exposing anything sensitive?!

[droidmonkey]

OK, I added support for setting a database icon (choosing from the set of default icons only):

[droidmonkey]

@phoerious whatcha think?

[RJVB]

OK, I added support for setting a database icon (choosing from the set of default icons only):

Looks a lot better, and as I thought the DB icon should be a great help. Is there a structural reason why you only support default icons, or is that just a "let's start with that and leave custom icons for later"?

[droidmonkey]

The difference is between storing a number outside the encrypted data and storing a binary blob which constitutes the custom icon.

[RJVB]

The difference is between storing a number outside the encrypted data and storing a binary blob which constitutes the custom icon.

So it should be technically possible to extend this by adding a way to define "custom default icons", at some point. I don't want to make a big deal out of this but I do think that being limited to a selection of default icons would quickly annoy me...