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

does not work with pages without title tag #79

Closed
ataiger opened this issue Apr 19, 2017 · 6 comments
Closed

does not work with pages without title tag #79

ataiger opened this issue Apr 19, 2017 · 6 comments
Assignees

Comments

@ataiger
Copy link

ataiger commented Apr 19, 2017

Tab Modifier version

0.19.2

Expected behavior

work with pages without <title> tag

(on these pages, chrome displays the url. what I want to do is to change that.)

Actual behavior

does not work

Steps to reproduce the behavior

thanks for nice work!

@sylouuu
Copy link
Collaborator

sylouuu commented Apr 19, 2017

Hi @ataiger,

TM is HTML-based. There is no way to change a tab title without HTML (as I know). I'll add this known issue in the README.

Best

@sylouuu sylouuu self-assigned this Apr 19, 2017
@ataiger
Copy link
Author

ataiger commented Apr 19, 2017

thanks for quick response.

so the extension can change the content of the title tag, but cannot create a title tag if the page didn't have one in the first place?

@sylouuu
Copy link
Collaborator

sylouuu commented Apr 19, 2017

I wondered if you talked about an HTML page without <title> tag or a non-HTML page?

Can you give me a link please?

@ataiger
Copy link
Author

ataiger commented Apr 19, 2017 via email

@sylouuu
Copy link
Collaborator

sylouuu commented Apr 19, 2017

I found why the title is not applied. At load, the title is an empty string because the website is loading dynamically. If I remove the empty string check it works.

I'll release a patch in few hours, be sure to opt-in the notification in TM settings to know when TM is updated.

Best

@ataiger
Copy link
Author

ataiger commented Apr 19, 2017 via email

@sylouuu sylouuu closed this as completed Apr 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants