Skip to content

Fixes for CSSMediaRule (#360) - #362

Merged
rbri merged 3 commits into
HtmlUnit:masterfrom
fdanek:cssmediarule_fix
Jun 30, 2021
Merged

rbri merged 3 commits into
HtmlUnit:masterfrom
fdanek:cssmediarule_fix

Conversation

@fdanek

@fdanek fdanek commented Jun 30, 2021 •

Copy link
Copy Markdown
Contributor

For issue #360:
MediaList default toString() now prints the media text.
CSSGroupingRule.cssRules, .insertRule() and .deleteRule() added (for
CSSMediaRule).
CSSConditionRule.conditionText added (for CSSMediaRule).

MediaList default toString() now prints the media text.
CSSGroupingRule.cssRules, .insertRule() and .deleteRule() added (for
CSSMediaRule).
CSSConditionRule.conditionText added (for CSSMediaRule).
@fdanek fdanek changed the title Fixes for CSSMediaRule (issue #360) Fixes for CSSMediaRule (#360) Jun 30, 2021
@fdanek
fdanek marked this pull request as ready for review June 30, 2021 12:36
@rbri

rbri commented Jun 30, 2021

Copy link
Copy Markdown
Member

can you please rebase - and solve the conflicts

@JsxSetter({FF, FF78, IE})
@JsxSetter(IE)
public void setCssText(final String cssText) {
rule_.setCssText(cssText);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think the IE ignores the setter like all the other browsers

@rbri
rbri merged commit 30838da into HtmlUnit:master Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants