Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Weird unicode text on page title #875

Closed
ghost opened this issue May 18, 2016 · 18 comments
Closed

Weird unicode text on page title #875

ghost opened this issue May 18, 2016 · 18 comments

Comments

@ghost
Copy link

ghost commented May 18, 2016

Visiting experiment detail pages show weird characters in the title bar. Screenshot @ https://www.evernote.com/shard/s3/sh/bf3f2560-af38-42e2-ae87-83246ca9c3e7/ee84a96f17b0ae021559351109aabd56

This is in Nightly on Linux, although I think Nick runs windows. The original report is bwinton/TabCenter#144

@ghost
Copy link
Author

ghost commented Oct 12, 2016

Experiment names are no longer in the titles, so, I guess this is fixed-ish

@ghost ghost closed this as completed Oct 12, 2016
@ghost ghost removed the status: ready label Oct 12, 2016
@ghost
Copy link
Author

ghost commented Jan 26, 2017

Experiment titles are back now and so is this bug...

@ghost ghost reopened this Jan 26, 2017
@ghost ghost added the status: ready label Feb 16, 2017
@mathieuinottawa
Copy link

Looks different to me, but still not right:

odd chars

@ScottDowne
Copy link

I noticed while the page is loading, it's fine. Then it hits setPageTitleL10N https://github.com/mozilla/testpilot/blob/master/frontend/src/app/containers/ExperimentPage.js#L214 and it updates the title, creating the issue. I think.

So maybe this is being caused by whatever is doing l10n?

@ScottDowne
Copy link

I think this and #1828 are related.

If you go here: https://github.com/l20n/l20n.js/blob/master/docs/html.md

At the bottom it says "If you care about IE11 you'll need the HTMLTemplateElement polyfill. L20n.js uses HTMLTemplateElements () to safely set contents of DOM nodes."

So maybe the weird characters are the broken template elements? They offer a solution which havn't looked into yet.

@ghost
Copy link
Author

ghost commented May 1, 2017

Yeah, interesting, although I can reproduce the page title ones in Firefox. /cc @stasm for the potential for an l20n tie-in

@ScottDowne
Copy link

Yeah, I think it's very likely l20n.js related, but I'm less sure it's <template> related.

I tried a few browser/OS combinations until I found one that failed (Windows + IE 11), I didn't find a FF that failed. I'm on Fedora (stable + nightly), and I also tried WIndows (stable and beta FF).

However, I also tried a template polyfil, and it didn't fix it.

I think a reduced test case to confirm this is in l20n.js and not something in test-pilot, would probably be the next thing I try.

@stasm
Copy link

stasm commented May 2, 2017

L20n (and Fluent) use Unicode's FSI and PDI marks to isolate the direction of interpolated text. This is useful when the text around an interpolation has a different direction than the text inside of the interpolation. The bidi algorithm needs a bit of help to properly render punctuation marks in those cases.

Unfortunately, FSI and PDI don't render well on Windows on some browsers. Is this issue only reproducible on IE11? Do you know if Edge is affected?

It might be best to turn off this behavior in l20n.js for browsers that are known to not support FSI and PDI well.

@ScottDowne
Copy link

@stasm Yeah, Edge is fine.

@stasm
Copy link

stasm commented May 2, 2017

Thanks for the information. I filed l20n/l20n.js#178. I'll be able to test it on Thursday.

@ghost
Copy link
Author

ghost commented May 2, 2017

I'm still reproducing this in Firefox Nightly on Ubuntu (and have been able since at least when this was filed). No one else sees this in Nightly's title bar (screenshot in first comment)?

Edit: The original screenshot was from Nick, not me. So, attaching what I see today:
nightly

@ScottDowne
Copy link

@wresuolc Yeah, it's fine for me on Fedora + nightly.

I'll see if I can wrangle a way to test on Ubuntu.

@ScottDowne
Copy link

I tested firefox and nightly on Ubuntu, and it didn't see this issue :(

I also tested l20n/l20n.js#178 best I could and could see a difference in IE 11.

@meandavejustice
Copy link
Contributor

Ubuntu, xmobar/xmonad, Firefox 54.0a2

Same behavior as @ScottDowne describes above. Title is fine until page is finished loading, then it changes.

weird-unicode-title

@ScottDowne
Copy link

Ah yes, interesting, for Linux it's probably more related to the status bar/desktop manager than the browser itself.

@ghost ghost removed the status:insprint label Jun 23, 2017
@ghost ghost added the status: ready label Jun 23, 2017
@johngruen
Copy link
Contributor

ref #1714

@ghost ghost added status: planned and removed status: ready labels Jul 10, 2017
@ghost ghost added this to the Stretch milestone Sep 18, 2017
@meandavejustice
Copy link
Contributor

This appears to be fixed now, probably in #1714 woohoo!

whatever

@SoftVision-PaulOiegas
Copy link

SoftVision-PaulOiegas commented Dec 11, 2017

@meandavejustice This still occurs on experiments details page on Windows 10 Edge browser.
image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants