-
Notifications
You must be signed in to change notification settings - Fork 48
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
GM_config.open(); GM_config.close(); GM_config.open(); causes duplicate settings outside of iframe #9
Comments
This is an edge case scenario where overriding onClose with another GM_config.open(); to prevent Save button from closing block level element. |
I don't really understand this issue. This sounds like improper usage and doesn't seem like something we should worry about. I'll give you the opportunity to explain this further and what you think should be done before I decided to close this issue or not. |
It's pretty simple... init... then open then close then open it again... the point that I've been typing all day is that Save closes the dialog... I'm almost ready to start a ticket on that one because the Cancel button should close and the save button is misleading by saving and closing. |
the save button is misleading by saving and closing.Yes, I guess it doesn't make sense for the save button to also close the window. And even if people wanted that they could use the onSave callback function to close the window after saving. This is the root issue. |
hmmm now I have to figure out why when I toggle open, close, open, close ... it stays closed forever :\ WIll look into my code a little deeper. |
I might know. It's probably an issue with toggling visibility. |
I've confirmed that this issue is still happening. on open(), close(), open() |
I'm not. This commit might have fixed it 73ca6d4 |
hmmm nope... I do an init Then programatically do an open on a test case... and if the user selects something different that isn't compatible with it I close it... then if the user selects something that is compatible it never reshows. |
Wait a second might be my conditional... ARGH about dain bread on this today. Hmmm just removed my conditional altogether... still same issue... lemme do some more digging. |
Yah it's my conditional. We're okay on this issue. I'll just have to change some logic. Although this is weird... is open(); open(); open() supposed to create three total instances of GMC? |
is open(); open(); open() supposed to create three total instances of GMC?No, definitely not. There is a check against this in open() for iframes. I just need another check for an html element. Edit: fixed in this commit 8dbc9a6 |
Hmm this last commit busts my logic from prior commit which was working Ughh... digs for a while There we go... GOT IT! |
busts my logic from prior commitSounds like your original solution was fragile. It wasn't that big of a change. |
Considering the volatile nature of GMC right now... it's no wonder my code busted ;) I've had to make a lot of changes today and dummy me put it a working copy of an existing script instead of a Unit Test... although that's still hanging around. :) |
it's no wonder my code busted ;) You're right, its pretty volatile right now. I've still gotta update my Facebook script in some places for the latest changes. For testing I actually symlink the @required file to the gm_config.js file in my git directory. |
I'm linking to the actual raw commit hash that way I know if there are any changes down the line it won't break my user.js |
causes duplicate settings outside of iframe.
The text was updated successfully, but these errors were encountered: