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.

Sign up for GitHub

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

Jump to bottom

Use a single property in Local Storage instead of many #176

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use a single property in Local Storage instead of many #176

wants to merge 2 commits into from

Conversation

CertainPerformance
Copy link
Collaborator

Fixes #175

@CertainPerformance
Use a single property in Local Storage instead of many
0164d31 
Fixes Benjol#175
@CertainPerformance
Save stored comment index with each <li>
4a649e5 
Previous version sometimes did not identify the index in the comment array correctly
MdoubleDash
MdoubleDash suggested changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@MdoubleDash MdoubleDash left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since src is no longer maintained (no new builds in foreseeable future), please consider making your changes on https://github.com/Benjol/SE-AutoReviewComments/blob/master/dist/autoreviewcomments.user.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MdoubleDash MdoubleDash MdoubleDash requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Can the userscript's Local Storage data be put into a single property, rather than 30 or more separate ones?
2 participants
@CertainPerformance @MdoubleDash