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

[Bug]: very long "long name" #4182

Open
jranma opened this issue Jun 26, 2024 · 7 comments
Open

[Bug]: very long "long name" #4182

jranma opened this issue Jun 26, 2024 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@jranma
Copy link

jranma commented Jun 26, 2024

Category

Other

Hardware

Not Applicable

Firmware Version

2.3.13

Description

image

Some user added many "/n" to his "long name", which causes display problems on every app.
Shouldn't these special control characters be filtered?

Relevant log output

No response

@jranma jranma added the bug Something isn't working label Jun 26, 2024
@garthvh
Copy link
Member

garthvh commented Jun 26, 2024

IMG_2774

I would have to allow them to work as line breaks on iOS

@Cisien
Copy link

Cisien commented Jun 29, 2024

The issue you're seeing has nothing to do with /n (which is just two normal characters and not a special escape sequence)

I tried reproducing by setting the name using several variations via the python cli with no impact on how the nodes table rendered.

image

image

image

@garthvh
Copy link
Member

garthvh commented Jun 29, 2024

They are newlines

https://unicode-explorer.com/c/2424

@garthvh
Copy link
Member

garthvh commented Jun 29, 2024

You can easily replicate this with
meshtastic --set-owner '''tbeamer Line1
Line2
Line3'''

@Cisien
Copy link

Cisien commented Jun 29, 2024

\n is a new line, /n is two regular ascii characters
and yes, i can replicate this on my nodes (images in the discord channel)

@garthvh
Copy link
Member

garthvh commented Jun 29, 2024

@thebentern I think we need to replace \n with a space

@Cisien
Copy link

Cisien commented Jun 29, 2024

may want to do the same with other characters. \t for example does some "interesting" things
image
in the IOS app it just looks like extra spaces

--set-owner "Cisien $(echo -e '\t\t')Rak"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants