-
Notifications
You must be signed in to change notification settings - Fork 17
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
Allow arbitrary rules of all circumstances #79
Comments
This is borderline out of scope for the extension. The purpose of the extension is to allow users to view content that can potentially be displayed inline but somehow is not. What you are effectively doing is to convert something that can already be displayed inline, to an alternative that does not correctly render the content. Based on your comment + error message at greasemonkey/greasemonkey#3164 (comment), it looks like you are mainly looking for a work-around for your user script to work around the stricter parser requirements in XML documents. Instead of that, I suggest to convert your input to well-formed X(HT)ML (e.g. using the For documentation on DOMParser, see https://developer.mozilla.org/en-US/docs/Web/API/DOMParser. |
On Sun, 02 Jul 2023 07:35:47 -0700 Rob Wu ***@***.***> wrote:
This is borderline out of scope for the extension. The purpose of the
extension is to allow users to view content that can potentially be
displayed inline but somehow is not. What you are effectively doing
is to convert something that can already be displayed inline, to an
alternative that does not correctly render the content.
Thank you for the elaborated answer.
Based on your comment + error message at
greasemonkey/greasemonkey#3164 (comment),
it looks like you are mainly looking for a work-around for your user
script to work around the stricter parser requirements in XML
documents. Instead of that, I suggest to convert your input to
well-formed X(HT)ML (e.g. using the `DOMParser` API with type
`text/html`) and then append the result (which should now be accepted
by the document because serializing the result again produces
well-formed markup).
That's correct.
I'm looking for a workaround, but not for the stricter parser
requirements as it is now solved.
A more important issue is the lack of functionality, such as `click()`,
which is resulted due to `document.contentType` being manifested as XML.
|
On Mon, 03 Jul 2023 03:49:41 -0700 Rob Wu ***@***.***> wrote:
Closed #79 as not planned.
I agree.
After more thinking, it does seem unfit, yet not so to those who are
willing manually edit plain text files, so it might still be a good
addition.
I'have also referred to
FirefoxBar/HeaderEditor#252 in order to
provide the experience that Open in Browser provides.
In any case, Open in Browser does allow my program to function in a
reasonable manner, and is the only solution I have, currently.
Hence it is the only solution available in the program.
Open in Browser is a good idea and should be a built-in feature in all
web browsers.
|
The following set of rules won't have any effect on
application/xml
andtext/xml
because both are opened in browser.The rules will work for
application/atom+xml
,application/rss+xml
etc.Please allow setting arbitrary content-type, even to content-types that already are opened in browser.
I need this feature in order for my program to work. See greasemonkey/greasemonkey#3164 (comment)
The text was updated successfully, but these errors were encountered: