Skip to content

Fix error with let and conditional block in FX 44 #1

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
Open
wants to merge 125 commits into
base: master
Choose a base branch
from
Open

Fix error with let and conditional block in FX 44 #1

wants to merge 125 commits into from

Conversation

oeekker
Copy link

@oeekker oeekker commented Nov 19, 2015

I'm not sure if this is the main project, because the add-on itself point to Erik Vold, but I cannot find the repository there anymore. Anyway, I've applied this patch locally and I can use this add-on again in FX and TB 44…

erikvold and others added 30 commits November 6, 2010 00:43
@erikvold
adding a access key (R)
a17391b
@erikvold
since startup is only calling findWindowsAndRun atm startup now does …
1ee6d81 
…what findWindowsAndRun used to do.
@erikvold
minor style change
0009b73
@erikvold
Closes johan#1 adding a accel + alt + R hotkey
c815bd3
@erikvold
removing useless var
97f0f5e
@erikvold
Closes #2 Supporting Seamonkey
dbabf5d
@erikvold
<version>2</version>
5e10c83
@supahgreg
add app (Firefox button) menu item
89c876d
@supahgreg
add an icon to the app menu item
ad31fa1
@supahgreg
mentioning myself
830e6e2
@supahgreg
changing icon to match source in install.rdf
5e85ad7
@supahgreg
removing image region, not needed with the refresh_16 icon
1c281ac
@supahgreg
SeaMonkey does not have the app menu (yet)
b4c142e
@supahgreg
use .cloneNode to create the app menu item
cfd3c61
@erikvold
Closes #5 inserting the restart menu item into Windows 7's app menu
c9079ae 
Merge branch 'issue-5'
@erikvold
saw a warning that the install() method was not defined, so I'm addin…
59e0642 
…g this and uninstall().
@erikvold
adding a shortcut for doc.getElementById(id);
2e5c5a2
@erikvold
minor comment changes
75cf102
@erikvold
using doc.createElementNS instead of doc.createElement
336538e
@erikvold
version 3
abc3071
@erikvold
removing some white space
f6314ec
@erikvold
minor style change
851ffbe
@erikvold
performance tweak: when shutting down the application, there is no re…
ebb0060 
…ason to do any cleanup.
@supahgreg @erikvold
Closes #8 allow users to change the hotkey
0a3bd62
@erikvold
Closes #10 using local icon
6faa5b4
@nmaier @erikvold
Add initial Readme
3c072ef 
(cherry picked from commit 802d8cd)
@erikvold
separating common utils from code unique to this addon
3bdfc0e
@nmaier @erikvold
Closes #11 Use local icons
ea59f3f
@nmaier @erikvold
add contributor
2025ec3
@erikvold
moving files needed in the xpi in to the 'src' dir
0333dae
erikvold and others added 28 commits June 5, 2011 10:52
@erikvold
Merge branch 'master' of github.com:erikvold/restartless-restart-ffext
dbc12da
@erikvold
adding extensions.restartless-restart.locale pref to options.xul
06bf817
@supahgreg
minor fix: typo in var name
0f021c8
@erikvold
using a better include() function
75cc863
@erikvold
adding a requre() method for commonjs style packages, and creating a …
09c3ace 
…unload package.
@erikvold
converting utils.js to commonjs style
b0f90a5
@erikvold
FF max ver is 8.* Fennec max ver is 6.*
be84b4b
@xabolcs
including options.xul in build process
285ed35
@erikvold
Merge pull request #46 from xabolcs/master
b03e503 
Inlined options is almost ready - options.xul should be included
@xabolcs
rrTBBB is undefined (regression by resolving #27)
187af60
@xabolcs
including packages in build process
92d1b7c
@erikvold
Merge pull request #39 from supahgreg/prefChgTypo
ba94844 
minor fix: typo in var name
@erikvold
Merge pull request #48 from xabolcs/master
02a91e4 
commonjs style is almost ready - packages should be included
@erikvold
ignoring *.komodoproject files
4c9a622
@erikvold
Closes #45 fixing error in Fennec
71a3468
@erikvold
Closes #44 uping max supported version numbers
7ca3e6d
@erikvold
<version>8</version>
217fba8
@erikvold
<version>9pre</version>
ded4c0a
@Morac2
Update src/packages/unload.js
c1f528f
@Morac2
Update src/packages/unload.js
7969f84
@erikvold
Merge pull request #57 from Morac2/master
d993067 
Fix for window unload issue r=@erikvold
@erikvold
<version>9</version>
16fbe9d
@erikvold
Delete Readme.md
2b305e0
icon remix for new firefox ui
32f65a9 
Improved icons for a better mix with the new firefox ui
@erikvold
Merge pull request #2 from andreldm/master
3d2b07b 
icon remix for new firefox ui
@oeekker
Rename restart.ak to restart.key, add restart.accesskey and add Dutch…
ad94e2b 
… locale
@oeekker
Add newline to last line
08dc6e9
@oeekker
Fix error with let and conditional block in FX 45
60378fb
johan
johan reviewed Nov 19, 2015
View reviewed changes
LICENSE.txt
Copy link
Owner

Choose a reason for hiding this comment

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

@erikvold, this technically just being a fork of your old/ex-project restartless-restart-ffext I forked at some point, in case the origin would disappear :) – @oeekker seems to have taken to updating it to something working today here, and added a nice MIT license file. Is that cool with you?

@johan
Copy link
Owner

johan commented Nov 19, 2015

Neat effort! I indeed haven't been more than forking this in case it'd turn out useful at some point and go lost, but I'm happy to merge an update, if Erik's cool with it. :)

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@oeekker @johan @erikvold @supahgreg @nmaier @xabolcs @Lakrits @drellmark @SamB @Morac2