Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When First Name starts with White Space, the card is Filed at the End, after Z #195

Open
7 tasks done
DiagonalArg opened this issue Nov 30, 2024 · 3 comments
Open
7 tasks done
Labels
bug Something is not working niche Only relevant to a very small amount of people

Comments

@DiagonalArg
Copy link

DiagonalArg commented Nov 30, 2024

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

v.1.1.0

Affected Android/Custom ROM version

GrapheneOS

Affected device model

Pixel 7

How did you install the app?

GitHub releases

Steps to reproduce the bug

Create a contact with a single space before the first name.

Expected behavior

Sock Android Contacts ignored the white space, which is probably the right solution.

Actual behavior

I just noticed that a contact with a (mistaken) single space before the first name was imported, it was sorted all the way at the end, after "Z".

(Tangentially, is there a vCard linter available anywhere? Answers may be offered here.)

Screenshots/Screen recordings

No response

Additional information

No response

@DiagonalArg DiagonalArg added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Nov 30, 2024
@Aga-C
Copy link
Member

Aga-C commented Nov 30, 2024

Please fill out the template properly.

@Aga-C Aga-C added the waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. label Nov 30, 2024
@DiagonalArg
Copy link
Author

Well I did it, but for a 2-line bug report, this kind of rigidity seems to me unnecessary.

@Aga-C Aga-C removed waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. needs triage Issue is not yet ready for PR authors to take up labels Dec 1, 2024
@Aga-C
Copy link
Member

Aga-C commented Dec 1, 2024

Create a contact with a single space before the first name.

It applies only to VCF files.

@Aga-C Aga-C added the niche Only relevant to a very small amount of people label Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working niche Only relevant to a very small amount of people
Projects
None yet
Development

No branches or pull requests

2 participants