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

Checkbox for "Add Scroll To Top button" seems amiss #366

Closed
IzzySoft opened this issue Sep 28, 2018 · 17 comments
Closed

Checkbox for "Add Scroll To Top button" seems amiss #366

IzzySoft opened this issue Sep 28, 2018 · 17 comments
Assignees
Labels
Milestone

Comments

@IzzySoft
Copy link

Installed Version:

2.3.8 DEV Environment: undefined

Current Behaviour

Looks line the check box for "Add Scroll To Top button" has disappeared:

screenshot

That makes it impossible to switch this feature off. Not that I want to, but others might 😉


SOX Errors logged to the browser console -- F12 (if available)

unlikely to be relevant, the only line I found:

SOX:  inlineEditorEverywhere error: sox.Stack.using not found

Steps to reproduce

Just click the icon in the MetaCollider™ to open SOX settings.


Features Enabled

Again subject to change, just started going through the settings one by one:

["Appearance-alignBadgesByClass","Appearance-colorAnswerer","Appearance-displayName","Appearance-dragBounty","Appearance-highlightQuestions","Appearance-isQuestionHot","Appearance-localTimestamps","Appearance-metaChatBlogStackExchangeButton","Appearance-scrollToTop","Appearance-spoilerTip","Appearance-tabularReviewerStats","Appearance-topAnswers","Appearance-unspoil","Appearance-addTimelineAndRevisionLinks","Appearance-showTagWikiLinkOnTagPopup","Comments-autoShowCommentImages","Comments-commentReplies","Comments-commentShortcuts","Comments-confirmNavigateAway","Comments-copyCommentsLink","Comments-moveBounty","Comments-hiddenCommentsIndicator","Comments-onlyShowCommentActionsOnHover","Editing-addSBSBtn","Editing-editComment","Editing-editReasonTooltip","Editing-findAndReplace","Editing-kbdAndBullets","Editing-titleEditDiff","Editing-inlineEditorEverywhere","Editing-pasteImagesDirectly","Flags-flagOutcomeTime","Flags-flagPercentages","Flags-flagPercentageBar","Sidebar-hideCommunityBulletin","Sidebar-hideJustHotMetaPosts","Sidebar-hideHireMe","Sidebar-hideChatSidebar","Sidebar-hideLoveThisSite","Sidebar-hideHotNetworkQuestions","Sidebar-linkedToFrom","Chat-chatEasyAccess","Chat-replyToOwnChatMessages","Chat-renameChat","Voting-betterCSS","Voting-stickyVoteButtons","Voting-disableVoteButtons","Extras-alwaysShowImageUploadLinkBox","Extras-linkedPostsInline","Extras-shareLinksPrivacy","Extras-shareLinksMarkdown","Extras-sortByBountyAmount","Extras-warnNotLoggedIn","Extras-hideCertainQuestions","Extras-showMetaReviewCount","Extras-copyCode","Extras-dailyReviewBar","Extras-openLinksInNewTab"]
@shu8
Copy link
Member

shu8 commented Sep 29, 2018 via email

@shu8 shu8 self-assigned this Oct 6, 2018
@shu8 shu8 added the bug label Oct 6, 2018
@shu8 shu8 added this to the 2.4.0 milestone Oct 6, 2018
@shu8
Copy link
Member

shu8 commented Oct 6, 2018

@IzzySoft Please could you right click on the add scroll to top button and click 'inspect element' and screenshot what comes up? I can't reproduce it :/

@IzzySoft
Copy link
Author

IzzySoft commented Oct 6, 2018

Yes, I can:

HTML

Looks like the checkbox is hidden. So, one more screenshot:

HTML with CSS

See: display: none !important. If that wasn't you, it must collide with some other UserScript or UserStyle. Though the name of the rule looks weird. Clicking on the rule points to "inline stylesheet", with a bunch of rules from SOUP again – but I couldn't find that rule there. Couldn't even make it show by overriding CSS on the element itself (display:block visibility:visible, both with !important) – which seems very strange. Disabled SOUP and reloaded – no change. Disabled all other UserScripts – nada. Disabled all UserStyles – niente. Out of ideas what I could try from my end now.

@shu8
Copy link
Member

shu8 commented Oct 6, 2018

@IzzySoft Thanks for the thorough debugging! I can't understand what's happening here 😆 Does this happen on a specific site, or is the checkbox hidden in all of them?

Please could you paste the entire block of styles that clicking the weird rule takes you to? I could try seeing if there's any SOX rule within there!

@shu8
Copy link
Member

shu8 commented Oct 6, 2018

@IzzySoft Also, for the other error you got, just underneath it there should be a inlineEditorEverywhere: Stack object: followed by another line of the object -- could you paste that here too please? :)

@IzzySoft
Copy link
Author

IzzySoft commented Oct 6, 2018

Does this happen on a specific site, or is the checkbox hidden in all of them?

All I've checked. Need to check at work on Monday if the issue is present there as well; I vaguely remember it is not, and the difference would be the token being present there.

Please could you paste the entire block of styles that clicking the weird rule takes you to?

Sure: https://pastebin.com/i1Eht1Mv (was too fooking long; what the heck is that long list of URLs there?)

just underneath it there should be a inlineEditorEverywhere: Stack object:

Afraid not:

SOX:  inlineEditorEverywhere error: sox.Stack.using not found    sox.common.js:32:13
soup JS fixes applied

No more SOX after that. Just a new one before that:

SOX:  There was an error loading the feature "Appearance-colorAnswerer". Please raise an issue on GitHub, and copy the following error log:
TypeError: this.querySelector(...) is null   sox.common.js:38:13

Ugh. Another issue to open?

PS: Why are the buttons "Close and comment" and "Comment" just inverted compared to GitLab? Guess that's why you see close/reopen so often within a single minute … Almost happened to me now 🤣

@shu8
Copy link
Member

shu8 commented Oct 6, 2018

@IzzySoft Thanks! Those CSS rules really are weird! :/ It seems like it's hiding all social links and buttons and buttons like 'back to top', 'scroll to top' along with it. Do you have any extensions other than uBlock origin installed that hide 'social' buttons?


I think the 2nd debug line only comes if debugging is enabled -- could you check it is still enabled please?


Don't worry about opening another issue, I'll make sure I look into that too! Which page were you on when it happened?


I've never used gitlab so I didn't know that! Although, I don't remember seeing many close/reopens myself!

@IzzySoft
Copy link
Author

IzzySoft commented Oct 6, 2018

Do you have any extensions other than uBlock origin installed that hide 'social' buttons?

None that I'm aware of (just went through the list again). Only one CSS, but that's not active on SE sites (deactivated it nevertheless, no effect). But even if, I find it very strange that I can't even override that on the object itself (style="display:block …"). But true enough: as soon as I change the ID, the check box shows up. Sufficient to just add a letter to the end, like "ScrollToTopp".

I think the 2nd debug line only comes if debugging is enabled -- could you check it is still enabled please?

As the link at the end of settings say "Enable Debugging", I'd say it's not active. Wait: on that page the error is not present in console. I only seem to have that on Android.SE (link says the same there). Just verified here: https://android.stackexchange.com/q/29447/16575

Not from SOX, but I love this kind of lines:

Loading failed for the <script> with source “https://www.google-analytics.com/analytics.js”.

@IzzySoft
Copy link
Author

IzzySoft commented Oct 6, 2018

PS: Just disabled the mega-block in "inspect element". A few seconds later the check box showed up. So at least it's clear now which rule is responsible. Remains the question who establishes it.

Looks like it is uBlock Origin. I just grepped my entire FF profile for "hero-content" (first line of that mega rule). Only place it appears is in uBlock's SQLite database. And guess what, "ScrollToTop" is in there as well. OK, need to write down what rule sets I've included here and verify that with what I have at work … nope, not needed: "Fanboy’s Annoyance List" includes "#ScrollToTop". Checking: Disabled that list, Checkbox appears.

@shu8
Copy link
Member

shu8 commented Oct 7, 2018

@IzzySoft Nice! it's an interesting way it uses to hide it! Glad it's sorted though :)


Just to clarify -- is it the inlineEditorEverywhere error only happens on Android? If so, could you check SR too, as it may be to do with you being a mod there?

@IzzySoft
Copy link
Author

IzzySoft commented Oct 7, 2018

Got that inlineEditorEverywhere error also on SR, e.g. here: https://softwarerecs.stackexchange.com/q/52574/185

As for that check-box: If I need it I now know how to get to it. Still arguing with myself whether to disable that Fanboy list or not: this might be the first culprit I encountered, who knows if there are more. It's a bit crazy to block a simple ID like this without leaving a chance to counter that (I tried and failed). Checking what that list holds feels like 50:50 (half of it is useful, the other half crazy). Guess I kick it altogether.

shu8 added a commit that referenced this issue Nov 4, 2018
@shu8
Copy link
Member

shu8 commented Nov 4, 2018

@IzzySoft the inline editor everywhere issue should be fixed in the latest dev version :)

That check box thing is pretty annoying! If you could just edit the list and remove the go to top selector that would be the best because it's a pretty common button that can be useful sometimes!

@shu8 shu8 closed this as completed Nov 4, 2018
@IzzySoft
Copy link
Author

IzzySoft commented Nov 5, 2018

Thanks! That error was replaced now by

There was an error loading the feature "Editing-inlineEditorEverywhere". Please raise an issue on GitHub, and copy the following error log:

TypeError: sox.Stack.using is not a function

😱

And why the heck does the preview suddenly not work here at Github? Strange …

Editing that list won't do any good, as the rule would be back at the next update. A "counter rule" should work, or removing that list. Not urgent for me, though, so don't invest too much time in it 😉

@shu8
Copy link
Member

shu8 commented Nov 5, 2018 via email

@IzzySoft
Copy link
Author

IzzySoft commented Nov 5, 2018

Take your time – I just found this checking the console, so it doesn't seem really urgent 😉

As for the preview: it's broken for me on my second machine as well. If it's currently broken for you, too, I'll just wait and see tomorrow. A quick search didn't show anything, Github status says "all green".

@shu8
Copy link
Member

shu8 commented Nov 9, 2018

@IzzySoft I can't reproduce it :/ Which browser are you on again? (sorry, I keep forgetting!)

The preview was working for me! Is it working now?

@IzzySoft
Copy link
Author

No, and neither is edit and opening pull requests (in case I caused confusion: I'm talking about Github here, not about SOX). And I use Firefox/Waterfox.

shu8 added a commit that referenced this issue Feb 3, 2019
* new dev

* #305

* #307

* #310

* #310

* Better fix for #310

The old fix placed the SOX button to the right of the username, but the username only shows up in the top bar if you're logged in. This meant the SOX button would be absent if you were logged out. This new code should fix that.

* #310

* #309, #313

* #311, #312

* #256

* #319, #317, #316

* #256

* #318

* #294 deprecate enhanced editor

* #308, other small fixes

* #320

* minor fixes (#324)

change scroll top top feature JSON info

* fixes #329 (#330)

also reduces jQuery dependency

* v2.1.4 DEV before push to master

also adds #326

* v2.2.0

* inital DEV push

* update readme to state GM not supported; #306

* reduce dependency on `hotkeys`+minor details (#333)

* #334 fixed

* #334

* #334

* #338

* #335, #338

* Micro optimizations (#336)

* update source link

* micro optimizations

- converted jQuery .not to CSS selector for faster perf (https://stackoverflow.com/questions/8845811/performance-differences-between-using-not-and-not-selectors)
- introduced a separate function to eliminate thrice-repeated (!) code, to keep it DRY

* fixes some issues (#339)

described in #338 (comment)

* fixes #338

* fixes #338 (#340)

1. used addEventListener instead of onload
2. descriptive names (`reader`)
3. optimized extraction of the image URL from the `data`
4. made all declarations `var` and moved them into one place for consistency
5. shortened the POST url to simply `'/upload/image?https=true',` ;P

* #322 implemented

* Sticky property

* Update sox.user.js

* Updating button colors

The main SE buttons use a lighter color than the SOX buttons

* Update sox.css

* Minor fixes

Patched a problem with vote buttons, and fixed the color of the notify on edit button

* Update sox.user.js

* minor changes/fixes

* #341 temporary fix

* Updated mod diamond icon

Updated to the icon, to match the one used by the new topbar

* Updated diamond icon

* Updated mod diamond

* improvements in colorAnswerer (#344)

* improvements in colorAnswerer
1. constant variables+cache answeredID
2. reduce jQuery dependency+shorter a[href] selector via user ID
3. change coloration

* optimization: use filters throughout (#345)

reduces bandwidth consumption

* fixes a few features; adds CSS

Fixes moveBounty, dragBounty, copyCommentsLink.
Adds class for colorAnswerer.
Updates jQuery

* fixes quickAuthorInfo fontawesome icon

* fix a few of #308

* Major improvements in code extensibility, reduced API usage, eliminate redundancy, etc. (#348)

* fixes various bugs mentioned in #348

* fixes standOutDupeCloseMigrated for search pages

* remove unwanted console logs

* #347, various minor tweaks

* fixes small bug in parseCrossSiteLinks

e.g. on this comment: https://meta.stackoverflow.com/questions/370902/i-need-a-lot-of-help-to-write-a-query-for-the-stack-exchange-data-explorer-sh?cb=1#comment608548_370902

* Fixed Meta SE's chat link

Added a / to the end of Meta SE's if statement to fix it. Also replaced "discuss" with "meta" for Area 51 (to match the new URL for Area 51 Discussions), shortened a bit of code, and added some comments

* Fixed Meta SE links

* Changed topbar z-index

Changed the fixed topbar's z-index to match the one on SO, in order to prevent elements like the usercard (z-index: 1000) from covering it

* Made it clearer what the checkbox does

* Many fixes to dialog buttons

Fixes many of the bugs from #308

* Many fixes to dialog buttons

Fixes many of the bugs from #308

* Many fixes to dialog buttons

Fixes many of the bugs from #308

* Many fixes to dialog buttons

Fixes many of the bugs from #308

* Fixed Area 51 Discussions bug

* #348 minor tweak

* fix (#350)

...for answerer ID 13 also matching commenter ids beginning with 13 (1345, 13555, etc.)

* #352, #353 remove yahoo YQL use

* #352

* PR/351 (#355)

* update getQuestionTags

* provide immediate response to user

* minor changes

* #325: remove rangyInputs dependency (#354)

* bump version number

* remove downvotedPostsEditAlert; remove rangyinputs `require`; implement mini changelog in dialog

* install stale and no-response bots

* #356 added, minor bug fix

* start #349, start to move to es6

* onlyShowCommentActionsOnHover bug fix

* #357

* deprecate fixedTopbar feature for #349

* add showTagWikiLinkOnTagPopup feature

* #359 fixed

* #358 fixed

* fixes #360

* fixes for #361

* #361 changes

* v2.3.0 final

* 2.3.0 DEV

* fix for #363

* implemented 'feature packs' in settings dialog

* debugging for #363, fixes #364

* #363 debugging

* fix for #363

* #361 implemented (access token no longer mandatory); minor code cleanup

* #365

* fix #368

* Add #315

* fix #366

* start to use linting

* fix #372

* add #370

* clean up code

* implement #374

* fix #373

* add #370; deprecate pasteImagesDirectly

* #374 replace 'help' instead of prepend new link

* change indentation to 2 spaces

* lint

* implement #296

* implements #376 (linkify meta diamond dialog title)

* fix #375 (topAnswers post score not being found)

* improve customMagicLinks feature -- add button to help menu on all pages

* fix sticky vote buttons feature

* keep new meta posts under diamond even after clicking (#378)

* editComment improve design, add more defaults (#377)

* fix better css feature

* editComment: make dialog delete button smaller

* improve editComment for #377; allow editing; new setting storage format

* add delay to comment features for links to deleted comments #379

* bump to v2.4.0

* Remove accidental letter introduced into file

* Bump installation link version numbers

* Lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants