Comments (15)
Sure, this is what I see (apologies if I'm missing something, I'm definitely not a Javascript developer):
from refined-github.
This issue doesn't happen if the quick-comment-edit
feature is disabled.
from refined-github.
To those using Firefox, can you try using an older version of Refined GitHub? Like from November
quick-comment-edit
also caused performance issues, disabling it in the newest version fixed it for me.
Just tried version 23.11.3 released November 3th (with quick-comment-edit
enabled) and the performance issues are still there, but much less. At least it doesn't lock up the tab completely...
from refined-github.
Can you show what code appears when you click "Debug script"?
Related:
from refined-github.
@fregante I've added you as a collaborator on https://github.com/dakrone/label-test, where I've manually created 385 labels so far (I got that far before Github started hitting me with errors about rate-limits) there are now 500 labels, and the problem reproduces. In case it's helpful for debugging.
from refined-github.
Even with quick-comment-edit
disabled it can happen, but that's just incidental.
from refined-github.
Oh, I checked again, it still happens with quick-comment-edit
disabled.
from refined-github.
I now found out that you have to type slowly for it to work. So I'm guessing there's another reason why this happens. Any way we can deliver debug info of some sorts?
from refined-github.
@fregante is there a subset of refined-github that I can disable to avoid this in the short-term? I've had to disable it entirely since I do a lot of daily label changes.
from refined-github.
That's what I'm trying to figure out. I disabled those 4 features an I still have trouble opening long PRs in Safari.
from refined-github.
With quick-comment-edit
enabled:
With quick-comment-edit
, locked-issue
and wait-for-checks
disabled (these 2 are disabled via a hotfix on my normal firefox instance but were enabled in the dev env):
The code at quick-comment-edit
does say:
// TODO: Replace with :has selector
observe(preSelector + '.js-comment.unminimized-comment .js-comment-update', addQuickEditButton, {signal});
Is there maybe a performance reason why it should use :has
?
(lol, after posting this issue it seems I still have the quick comment edit button, so I guess disabling a plugin works differently in a dev env)
from refined-github.
quick-comment-edit definitely causes perf problems in firefox indeed. One very simple way for me to trigger the slowness is resizing the browser window a few times in a row. Once in a while the page will completely freeze and not redraw for multiple seconds.
from refined-github.
To those using Firefox, can you try using an older version of Refined GitHub? Like from November
I have a suspicion that the "slow" code is in the CSS, because I can't think of a reason why removeChild
would not be instant. I'd consider it a browser bug that affects RG users :(
from refined-github.
Are there any docs on how to revert to such version?
from refined-github.
You can download them from https://addons.mozilla.org/en-US/firefox/addon/refined-github-/versions/
from refined-github.
Related Issues (20)
- Preview README before file tree HOT 2
- Token validator fails with no `read:project` permission even if full `project` permission is given. HOT 1
- `swap-branches-on-compare` doesn't show swap button when comparing `child...parent` HOT 2
- Unable to add enterprise Github domain HOT 1
- Convert to GitHub Apps HOT 2
- Browser Notifications HOT 3
- `unreleased-commits`: draft release button wraps below
- `update-pr-from-base-branch` button should allow one to default to rebase HOT 3
- Add a quick menu for the milestone page HOT 1
- `quick-review` doesn't autofocus the field
- Filter `everything you're involved` bring me back to issues.
- I forgot to set the token HOT 2
- [Feature] Allow for a `--first-parent` kind of view of a branch history HOT 1
- Remove `clear-pr-merge-commit-message` HOT 1
- RFE: dim draft PRs in the PR list
- prevent-pr-merge-panel-opening doesn't work properly
- Show full committer and author details
- `github-actions-indicators`: indicator does not show up
- `clean-conversation-sidebar` breaks the labels picker HOT 5
- Button to list all commits in PR HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from refined-github.