Skip to content

Conversation

CrazyCraftix
Copy link

See #1691

Yeah, so I thought this would be easier - it's not done yet.

I used android.text.style.ReplacementSpan to render a gray "•" in place of spaces. This works really well everywhere except around line wraps - which were the whole reason I wanted this in the first place. This is the current state of the code (see first image). The main, almost trivial change for the feature itself is in fc43527. In the 2 commits before that I reworked the coloring code a bit - I believe for the better.

It's easy to get the looks right by swapping the underlying space character to something else and rendering that as gray dot (see right image). This is problematic because copying the text then has that other character and no spaces in it. You can easily fix this for the copy button, but I'm not sure about manual text selection and the cut/copy popups.

I'm going to keep working on this to try and find something that works. Until then, maybe someone else has a good idea or has some use for what I've done so far.

@github-project-automation github-project-automation bot moved this to In progress in KeePassDX 5.0.0 Jul 22, 2025
@J-Jamet J-Jamet moved this from In progress to To do in KeePassDX 5.0.0 Jul 22, 2025
@J-Jamet
Copy link
Member

J-Jamet commented Jul 22, 2025

It's always like this, we think it will be quick and it always takes longer than we think.

You need to check the copy from the clipboard popup, but also from the Magikeyboard and Autofill. As you say, I think we can get away with a copy button; I'll let you figure it out, as it's not an urgent feature.
I've added it for version 5.0.0, which will contain some graphical refactoring.

Thanks for your work :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To do
Development

Successfully merging this pull request may close these issues.

2 participants