-
-
Notifications
You must be signed in to change notification settings - Fork 52
Quick Reblog: Fix popup on accounts with new footer variant #1823
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
Quick Reblog: Fix popup on accounts with new footer variant #1823
Conversation
AprilSylph
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- No-features variant
- Confirm that the Quick Reblog modal appears when hovering over the reblog button.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled.
- Confirm that clicking the blog selector, text boxes, quick tags buttons, and submit buttons in the Quick Reblog modal only do what they're supposed to.
- Confirm that the Quick Reblog modal functions.
- Confirm that the reblog button changes color when Quick Reblog is used to reblog a post.
-
postFooterSplitNotesCount- Confirm that the Quick Reblog modal appears when hovering over the reblog button.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled.
- Confirm that clicking the blog selector, text boxes, quick tags buttons, and submit buttons in the Quick Reblog modal only do what they're supposed to.
- Confirm that the Quick Reblog modal functions.
- Confirm that the reblog button changes color when Quick Reblog is used to reblog a post.
-
postFooterSplitNotesCountSingleAction- Confirm that the Quick Reblog modal appears when hovering over the reblog button.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled.
- Confirm that clicking the blog selector, text boxes, quick tags buttons, and submit buttons in the Quick Reblog modal only do what they're supposed to.
- Confirm that the Quick Reblog modal functions.
- Confirm that the reblog button changes color when Quick Reblog is used to reblog a post.
Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled.
In the postFooterSplitNotesCountSingleAction variant, the reblog button is not disabled; instead, the popup that appears when clicking it has no reblog options, just "See reblogs" to open the notes. As a result, our code has no idea that it shouldn't show a Quick Reblog modal on the button. I guess we have to test the React data instead of relying on disabled/aria-disabled now? :ugh:
|
Oh, right, I discovered this but forgot that it would have implications for this PR: there's different behavior for "reblogs disabled; has no reblogs" (disabled button) and "reblogs disabled; has reblogs" (button is enabled and only links to the reblogs). |
|
Checking Edit: Hah, no, it doesn't fix this, because Not sure if there's another way yet. |
|
On all 3 footer variations:
Note that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post on a password-protected blog on the "channel" page, but does appear when doing so in the blog view modal and will 404 when used. This is a Tumblr bug, as the native reblog button will also be enabled and will also show an error message when clicked in this case. |
AprilSylph
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- No features variant
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled that has never been reblogged.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled but that has been reblogged before and thus has a reblog/note count.
-
postFooterSplitNotesCount- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled that has never been reblogged.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled but that has been reblogged before and thus has a reblog/note count.
-
postFooterSplitNotesCountSingleAction- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled that has never been reblogged.
- Confirm that the Quick Reblog modal doesn't appear when hovering over the reblog button of a post with reblogs disabled but that has been reblogged before and thus has a reblog/note count.
Description
This fixes an issue on a new slight variant of one of the post footers that's floating around where
clicks within the quick reblog popup would trigger the reblog button and open the reblog interface.the Quick Reblog popup wouldn't appear when hovering over the reblog button.Resolves #1822.
Testing steps
On all 3 footer variations:
Note that the reblog button will become highlighted when mousing over the Quick Reblog popup in one split notes variant but not the other.