-
Notifications
You must be signed in to change notification settings - Fork 28
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
Should we drop trailing ? in rel-urls when there's no query string? #177
Comments
False alarm! Not an emoji parsing issue. 🏠 is coming through fine, it's just attached to |
Is that merging the correct behavior? Should mf2py do it? @tantek @kevinmarks 🤷🤷🤷 |
The following HTML: <a href=/? rel=foo>bar</a> with URL: parses on php.microformats.io to: {
"items": [],
"rels": {
"foo": [
"https://example.com/"
]
},
"rel-urls": {
"https://example.com/": {
"text": "bar",
"rels": [
"foo"
]
}
},
"debug": {...}
} and on python.microformats.io to: {
"items": [],
"rels": {
"foo": [
"https://example.com/"
]
},
"rel-urls": {
"https://example.com/": {
"text": "bar",
"rels": [
"foo"
]
}
},
"debug": {...}
} Considering the original problem is no longer applicable I'm going to close this issue. |
I'm seeing an odd bug. Looking at https://python.microformats.io/?url=https%3A%2F%2Ftantek.com%2F&parser=html5lib , the
rel-urls
item forhttps://tantek.com/
hastext
""
, but it should be the emoji 🏠. Other emoji show up fine intext
eghttps://tantek.com/?skin=vtx
correctly has 📺 :Fwiw the 🏠 comes through fine in php-mf2, https://pin13.net/mf2/?url=https://tantek.com/
The text was updated successfully, but these errors were encountered: