-
Notifications
You must be signed in to change notification settings - Fork 131
New Extension: Addressbar Updater #1656
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a couple nitpicks
There was a problem hiding this 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()) { |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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
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
