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

Added title and description meta tags #72

Merged

Conversation

askvortsov1
Copy link
Sponsor Member

@askvortsov1 askvortsov1 commented Feb 13, 2020

Modified code to set the meta "title" attribute based on the current page's tag, or to "Tags" (translatable) for pages related to the tags extension.

@askvortsov1 askvortsov1 changed the title Added title meta element based off of tags page(s) Added title and description meta tags Feb 13, 2020
@askvortsov1
Copy link
Sponsor Member Author

With the changes made in f1c65ae, this fully covers requirements of #69 and #48, which can probably both be closed if this is good to merge.

@luceos
Copy link
Member

luceos commented Feb 24, 2020

This is a very decent PR @askvortsov1 well done!

Copy link
Contributor

@franzliedke franzliedke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thanks!

Just some questions, I might be misunderstanding things here...

js/src/forum/addTagFilter.js Outdated Show resolved Hide resolved
js/src/forum/addTagFilter.js Outdated Show resolved Hide resolved
@askvortsov1
Copy link
Sponsor Member Author

Addressed

@franzliedke franzliedke merged commit 61513f1 into flarum:master Mar 6, 2020
@franzliedke
Copy link
Contributor

Thank you!

@askvortsov1
Copy link
Sponsor Member Author

@franzliedke the accompanying flarum/lang-english#154 still needs to be merged

@franzliedke
Copy link
Contributor

I know. I will change the keys, though.

franzliedke added a commit that referenced this pull request Mar 6, 2020
franzliedke added a commit to flarum/lang-english that referenced this pull request Mar 6, 2020
askvortsov1 pushed a commit to flarum/lang-english that referenced this pull request Mar 11, 2022
askvortsov1 added a commit that referenced this pull request Mar 11, 2022
askvortsov1 pushed a commit that referenced this pull request Mar 11, 2022
askvortsov1 pushed a commit to flarum/lang-english that referenced this pull request May 10, 2022
askvortsov1 added a commit that referenced this pull request May 10, 2022
askvortsov1 pushed a commit that referenced this pull request May 10, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants