GithubHelp home page GithubHelp logo

soup's People

Contributors

gaurangtandon avatar manishearth avatar vyznev avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar

soup's Issues

New SO navigation: inbox and achievement boxes misaligned after AJAX load

When you opt-in to the new SO navigation (see https://meta.stackoverflow.com/questions/343103/help-test-the-new-top-nav-phase-3-testing-runs-this-week), the inbox and achievent boxes jump after ajax load completes:

Just to be clear: the box should not load empty in one location, then jump to another.

This only happens when SOUP is enabled in tampermonkey. When I disable SOUP, the boxes stay put.

Tested on macOS Sierra 10.12.2, Chrome 55.0.2883.95

SOUP is breaking Enter keyboard shortcut for comments

It seems SOUP is affecting the comment boxes breaking the Enter key. With it enabled, pressing Enter to submit or edit a comment doesn't work, nothing happens. Disabling fixes the issue.

Perhaps a fix similar to the one for http://meta.stackexchange.com/a/285035 is needed. Michael mentions something about changes to the :visible selector.

SOUP 1.45.22 (via Tampermonkey 4.1.10)
Stack Overflow rev 2016.10.5.4036
Win 7 SP1
Chrome 53.0.2785.143

Reduce the time to load spoiler text as it used to be originally

I find the implementation of the placeholder text ("spoiler: hover/click to reveal") a bit unproductive. With SOUP enabled, I've to wait a bit longer to view what's inside the spoiler. First the placeholder text ("spoiler: hover/click to reveal") fades out, and only then the spoiler text fades in.

I am pretty sure that the time taken for the spoiler text to load is longer than it used to be without SOUP enabled. Is there a way to reduce the time to load spoiler text as it used to be originally (without SOUP)?

mse90713 fix (bounty message in close dialog) broke at some point

The mse90713 fix (which adds a warning to the close dialog on questions with active bounties indicating that they cannot be voted for closure) broke at some point, possibly due to SE design changes. (I can't pinpoint when exactly, since I've not run into a case where I've attempted to close a bountied question in a while.)

fixes.mso300679 broken due to changes rolled out on 2019-10-03

Today SE rolled out a change titled "Refactor post validation", causing the following error to appear in the console:

TypeError: StackExchange.postValidation.getSidebarPopupOptions is not a function -1:333:47
    <anonymous> https://stackoverflow.com/users/message/create/-1:333
    jQuery 3
    MarkdownEditor https://cdn.sstatic.net/Js/wmd.en.js?v=c7f5081cdae1:2
    e https://cdn.sstatic.net/Js/wmd.en.js?v=c7f5081cdae1:2
    t https://cdn.sstatic.net/Js/wmd.en.js?v=c7f5081cdae1:3
    n https://cdn.sstatic.net/Js/full.en.js?v=1d06422f5fb7:4

This is from fixes.mso300679:

var options = StackExchange.postValidation.getSidebarPopupOptions();

Unfortunately, this breaks the moderator-only 'contact user privately' feature, so I've had to disable S.O.U.P. altogether. This also breaks the WMD editor elsehwere, just not as completely; the editor is left in an unfinished state without buttons.

Request for new fix for pagination links for spam-deleted posts

I recently encountered a bug when viewing a post with multiple pages of answers that a moderator had locked and deleted, which had helpful spam or rude/abusive flags.

When trying to navigate to a subsequent page of answers using the pagination link, or when trying to access a specific answer that happens to be on a page other than the first page, the server-side logic that automatically removes the question title from the URL (to prevent it from being exploited by spammers) also causes things like ?page=2 to be stripped out, making it impossible to view a subsequent page unless I manually modify the URL to remove the question title while leaving the query parameter intact.

I've posted a detailed answer showing the cause of the bug and the workaround.

It would be nice to have the fix in SOUP. I think it's reasonable because all that needs to be done is remove the question title from the pagination links for deleted posts. (The "Return to Answer" links may not be fixable, but still.)

SOUP prevents adding comments to Teams posts

With SOUP enabled, I am unable to comment on posts in SO Teams. I get the following console errors:

TypeError: str is undefined[Learn More] 160:505:4

Error: Syntax error, unrecognized expression: #post-editor-/c/moderators160 jquery.min.js:2:12733

The format of a Team URL is https://stackoverflow.com/c/(team name)/questions/### -- the "/c" is the special marker here.

I've locally modified the script to operate on stackoverflow.com/questions (as opposed to *) to bypass the problem -- not ideal, but I don't know how to do "@match all except this pattern".

Restore tag popup?

Could you include a patch to restore the previous structure of the tag popup (with links to tag wiki etc, optionally also removing the two big buttons the new styling brought in)? For active users in general (and mods in special), those tag-wiki links are way more important than subscribing to or ignoring tags, so I feel the current design a big step backwards. Reference on MSO: Restore previous variant of tag popup.

Thanks in advance!

Spoilers no longer show "hover / click to show spoiler" text

The "hover / click to show spoiler" text on spoilers hasn't been working for quite a while now, probably because of SE changes a while ago. (I couldn't find which fix is the specific one for this, as there's only one that mentions spoilers in the list of fixes, and it's not related to this one.)

SOUP beta site detection is broken and should be replaced with a privileges API query

It appears that the CSS-sniffing hack used to set the SOUP.isBeta flag (which in turn is user to determine the rep threshold for various privileges) does not work (at least) on Chrome 67, causing the flag to always be false.

Fortunately, the only fix currently (as of SOUP v1.52) using this flag is mse115702. Thus, the only visible effect of this breakage is that delete links may not appear when they should when downvoting an answer to -1 on a beta site for users with at least 4k but less than 20k rep.

In any case, with the advent of design-independent graduation, the whole idea of this flag has become fundamentally misguided. The proper fix would be to replace it with an SE API user privileges query. The tricky part here is that we want to minimize the number of API queries made, which implies at least heavy caching of the results (e.g. in LocalStorage) and probably also on-demand fetching (with its attendant asynchronous complications).

The API query implementation from the Bounty Bar user script could presumably be used as a basis for this. It's rather bare-bones, but it seems to work well enough.

Small space is added between comments and timestamps

Small bug, but I thought I'd mention it. Normally, the comments look like this:
screen shot 2017-08-03 at 1 37 14 pm

However SOUP adds an extra space between the comment and the dash:
screen shot 2017-08-03 at 1 37 04 pm

This causes the timestamps to shift a bit when the page loads.

This breaks the new activity icon.

This breaks the new activity style. It should look like:

enter image description here

However it looks like this:

enter image description here

Removing the following style fixes this for me.

.question-summary .question-hyperlink {
  display: block;
}

Document future of Greasemonkey support

As I'm sure you're aware, Greasemonkey 4, due to release with FF 57, is not backwards compatible. Scripts will need to be adapted.

Please document which of the following options for Greasemonkey <4 users will be open regarding SOUP:

  1. SOUP will be ported to Greasemonkey 4.
  2. SOUP will no longer be support Greasemonkey; use Tampermonkey (or...?).
  3. SOUP will no longer be support Firefox as a userscript; use the browser extension.

SOUP causes Chat Easter Egg script to load twice in Edge

When using this script along with SOUP in Microsoft Edge, the dropdown it adds to the page shows up twice (or used to until the author of that script added a workaround). Disabling SOUP fixes it.

I reported this to the developer of that script, who responded by implementing a workaround that explicitly prevents their script from loading twice and displays a console error message if it attempts to load twice. With SOUP disabled, I don't see this console message; with SOUP enabled, I do. This is still a bug, because the script shouldn't be attempting to load twice with SOUP enabled in the first place.

Also, according to that script's developer, the problem does not occur in Chrome, so it could be a bug in Edge or in the implementation of the Edge port of Tampermonkey. I reported this here to investigate if this is indeed a problem in SOUP or not, since disabling SOUP fixes the issue.

Top bar is clipped on non-responsive pages on responsive sites, e.g. profiles on MSE

SE recently started deploying a new responsive design, but not all pages are responsive yet. Q&A is, but profiles and reviews aren't.

I recently noticed a problem with the top bar on those non-responsive pages and filed a bug report, but it turns out to be coming from something SOUP is doing. (I disabled user scripts one at a time until I found the one that introduced the change.) That bug report, with screen shots, is here: https://meta.stackexchange.com/q/311489/162102 . Basically, in a smaller window size (mine is 1036px wide), the right side of the top bar is clipped off only on these pages. On sites that don't have the responsive design yet (almost all of them) and responsive pages on SO and Meta.SE, the problem does not occur.

Eventually they're going to make everything responsive and presumably the problem will go away, but it's going to take them weeks to months to do that. Is there anything that can be done in the meantime? I have no idea how large or small an investigation/change this would be.

I really appreciate all the work you've put into SOUP! I'm sorry to come here asking for a fix instead of working it out and offering you a PR.

In case it matters, I'm using the userscript version, not the add-on.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.