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

Comms console ignores HTML tags #4348

Open
unid15 opened this issue May 8, 2015 · 11 comments
Open

Comms console ignores HTML tags #4348

unid15 opened this issue May 8, 2015 · 11 comments
Labels
Consistency Issue This thing in question isn't consistent across multiple things. Feature Request I think you should code this for me. Oversight Unintended consequence.

Comments

@unid15
Copy link
Contributor

unid15 commented May 8, 2015

@Exxion
Copy link
Member

Exxion commented May 8, 2015

NanoUI issue, probably.

@ComicIronic ComicIronic added the Bug / Fix This is either a bug or a fix for a bug. label May 8, 2015
@gbasood
Copy link
Contributor

gbasood commented Jan 18, 2016

Can't see original image not sure if still bug

@gbasood gbasood added the Needs Moar Testing This issue potentially doesn't actually happen the way it's stated to happen. / Test PR more. label Jan 18, 2016
@gbasood
Copy link
Contributor

gbasood commented Aug 29, 2016

@unid15 what did he mean by this

@Exxion
Copy link
Member

Exxion commented Aug 30, 2016

Damn it just upload the image to GitHub so it doesn't get deleted

@unid15
Copy link
Contributor Author

unid15 commented Sep 7, 2016

Not sure if this still exists, the bug was that HTML tags weren't rendered, so you saw this when looking at centcom's messages:

message here and here

instead of

message here and here

@gbasood
Copy link
Contributor

gbasood commented Oct 4, 2016

You mean in command reports as a captain? That's not a bug

@unid15
Copy link
Contributor Author

unid15 commented Oct 4, 2016

no there's a button on the console that lets you browse the received command reports or something like that, that's where the issue is

@PJB3005
Copy link
Contributor

PJB3005 commented Oct 4, 2016

Probably because the nano UI template is screwed up and enters the text literally instead of allowing the html to be parsed.

@Cyadox
Copy link
Contributor

Cyadox commented Oct 25, 2017

@unid15 @Shadowmech88
Honestly the html isn't parsed in the command report if it's a global announcementl, I think the only reason it is when it prints out is because paper objects parse HTML no matter what.

2 out of 3 places its stored, HTML is not parsed. Close the issue, yay or nay?
proofs

@Shadowmech88
Copy link
Contributor

Given that the HTML tags are only properly parsed by paper, and only because paper natively does so, I don't think this is a bug.
It could be re-labeled as a feature request, though.

@Shadowmech88 Shadowmech88 added Feature Request I think you should code this for me. Oversight Unintended consequence. and removed Bug / Fix This is either a bug or a fix for a bug. Needs Moar Testing This issue potentially doesn't actually happen the way it's stated to happen. / Test PR more. labels Oct 25, 2017
@Intigracy Intigracy added the Consistency Issue This thing in question isn't consistent across multiple things. label Oct 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Consistency Issue This thing in question isn't consistent across multiple things. Feature Request I think you should code this for me. Oversight Unintended consequence.
Projects
None yet
Development

No branches or pull requests

9 participants