Skip to content

Conversation

srsutherland
Copy link

Periodically updates the address bar with so that when refreshed, it returns to the last viewed post. Useful for tab crashes, resuming after restart, or with extensions like "The Great Suspender"

I'm not in love with the image, but I can't figure out how else to represent an address bar in a 64x64 square
adressbar

Copy link
Member

@AprilSylph AprilSylph left a comment

Choose a reason for hiding this comment

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

a couple nitpicks

Copy link
Member

@AprilSylph AprilSylph left a comment

Choose a reason for hiding this comment

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

there's no need to leave comments on things that are either self-descriptive or obvious in meaning if you read the rest of the code

},

update_address: function() {
if (!XKit.extensions.addressbar.focus_only || document.hasFocus()) {
Copy link
Member

Choose a reason for hiding this comment

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

XKit.extensions.addressbar.focus_only doesn't exist; you meant XKit.extensions.addressbar.preferences.focus_only

if (id !== null && id > 1) {
XKit.extensions.addressbar.replace_address(id);
}
return id; //unused/debug
Copy link
Member

Choose a reason for hiding this comment

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

always remove debug lines


update_address: function() {
if (!XKit.extensions.addressbar.focus_only || document.hasFocus()) {
//do nothing if you're not in focus
Copy link
Member

Choose a reason for hiding this comment

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

this comment should be removed too

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants