Skip to content
This repository has been archived by the owner on May 10, 2018. It is now read-only.

[GreaseMonkey] Userscripts not applied to iframe #1457

Closed
pejakm opened this issue Sep 16, 2014 · 7 comments
Closed

[GreaseMonkey] Userscripts not applied to iframe #1457

pejakm opened this issue Sep 16, 2014 · 7 comments
Labels

Comments

@pejakm
Copy link
Member

pejakm commented Sep 16, 2014

I'm not sure if this is a feature or limitation, but some scripts rely upon it.

Example: I made this script to apply GMail "Ocean" theme to RoundCube webmail (Login: demo1@demo.roundcube.net - demopwd).

How it looks in QupZilla: https://mediacru.sh/y-TDSqfbos64
How it look in Opera 12: https://mediacru.sh/spOEoWd6VwgR

@pejakm
Copy link
Member Author

pejakm commented Sep 16, 2014

Also, the script is not working in Settings, Identities and Responses tabs, and I can't figure out why, there's no report of any kind in console.

@nowrep
Copy link
Member

nowrep commented Sep 16, 2014

The link to script is not working (403). Can you please paste it somewhere?

@pejakm
Copy link
Member Author

pejakm commented Sep 16, 2014

Sorry, wrong file permissions. Fixed now, but here's a paste:
http://pastebin.com/62ezLBfB

@nowrep
Copy link
Member

nowrep commented Sep 16, 2014

Try it with GM_addStyle() function to add a style to page (instead of your code)

@pejakm
Copy link
Member Author

pejakm commented Sep 16, 2014

Not working. Plus, I doubt GM_addStyle is supported by Opera. However, I did notice the iframe was transparent for a short period of time, until the page loaded completely. Any thoughts?

@nowrep
Copy link
Member

nowrep commented Sep 16, 2014

The thing is, userscripts actually are applied to all frames, including iframes ...
But only when the main frame finishes loading, it does not track other frames.

@nowrep nowrep added the Bug label Sep 16, 2014
@nowrep nowrep closed this as completed in 171267c Sep 16, 2014
@nowrep
Copy link
Member

nowrep commented Sep 16, 2014

I still don't know why it doesn't work on those other pages.

nowrep added a commit that referenced this issue Sep 16, 2014
Fixes second part of  #1457 (not working on settings, identities, ..
pages)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants