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

significant improvement in microformats markup #1063

Merged
merged 13 commits into from
Apr 12, 2017
Merged

Conversation

dissolve
Copy link
Contributor

@dissolve dissolve commented Apr 6, 2017

This is a huge improvement and I believe will close #965.

Had these microformats reviewed by others in the community to help
ensure they are at least correct, if not complete.

I did not want to change the structure of the page, and so there it does
not fully mark up the entire ancestry chain, or reply chain, only the
direct decendants and direct ancestors are correctly associated, but
this is likely fine as the most important bit is to have access to the
urls for those toots which are now correctly fetchable.

Ben Roberts added 4 commits April 6, 2017 10:59
This is a huge improvement and I believe will close mastodon#965.

Had these microformats reviewed by others in the community to help
ensure they are at least correct, if not complete.

I did not want to change the structure of the page, and so there it does
not fully mark up the entire ancestry chain, or reply chain, only the
direct decendants and direct ancestors are correctly associated, but
this is likely fine as the most important bit is to have access to the
urls for those toots which are now correctly fetchable.
@Gargron Gargron added the ui Front-end, design label Apr 8, 2017
@raucao
Copy link
Contributor

raucao commented Apr 8, 2017

@dissolve Have you seen the CodeClimate failures? https://codeclimate.com/github/tootsuite/mastodon/pull/1063

(Not a maintainer here, just someone who likes the PR. 😉)

@dissolve
Copy link
Contributor Author

keeping this PR up to date with master, its a bit of a nuisance to keep updating it. Any idea what is needed to proceed here?

@Gargron
Copy link
Member

Gargron commented Apr 12, 2017

I'm reviewing this right now.


.status__content.e-content.p-name.emojify<
.status__content.p-name.emojify
Copy link
Member

Choose a reason for hiding this comment

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

The line is missing < at the end. That removes extra whitespace from the output HTML, which does get picked up visually because we want to support ASCII art in toot text

@Gargron Gargron merged commit 0254ee9 into mastodon:master Apr 12, 2017
thurloat pushed a commit to Moosetodon/mastodon that referenced this pull request Apr 13, 2017
* significant improvement in microformats markup

This is a huge improvement and I believe will close mastodon#965.

Had these microformats reviewed by others in the community to help
ensure they are at least correct, if not complete.

I did not want to change the structure of the page, and so there it does
not fully mark up the entire ancestry chain, or reply chain, only the
direct decendants and direct ancestors are correctly associated, but
this is likely fine as the most important bit is to have access to the
urls for those toots which are now correctly fetchable.

* improve code climate

* trying to pass code climate tests

* code climate

* fix p-summary for content warning posts

* fix error introduced when merging via github
Gargron added a commit that referenced this pull request Apr 13, 2017
Fix username styling regression introduced in #1063
Fix report screen background regression introduced in #1415
Gargron added a commit that referenced this pull request Apr 13, 2017
…#1699)

Fix username styling regression introduced in #1063
Fix report screen background regression introduced in #1415
Nyoho referenced this pull request in Nyoho/mathtodon Apr 25, 2017
* significant improvement in microformats markup

This is a huge improvement and I believe will close #965.

Had these microformats reviewed by others in the community to help
ensure they are at least correct, if not complete.

I did not want to change the structure of the page, and so there it does
not fully mark up the entire ancestry chain, or reply chain, only the
direct decendants and direct ancestors are correctly associated, but
this is likely fine as the most important bit is to have access to the
urls for those toots which are now correctly fetchable.

* improve code climate

* trying to pass code climate tests

* code climate

* fix p-summary for content warning posts

* fix error introduced when merging via github
Nyoho referenced this pull request in Nyoho/mathtodon Apr 25, 2017
…mastodon#1699)

Fix username styling regression introduced in #1063
Fix report screen background regression introduced in #1415
abcang added a commit to pixiv/mastodon that referenced this pull request May 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Front-end, design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

microformats clean-up
3 participants