Skip to content

chore: Setting style property to make CSP less grumpy.#7503

Merged
NeilFraser merged 2 commits intodevelopfrom
fraser-style
Sep 19, 2023
Merged

chore: Setting style property to make CSP less grumpy.#7503
NeilFraser merged 2 commits intodevelopfrom
fraser-style

Conversation

@NeilFraser
Copy link
Copy Markdown
Contributor

"Content-Security-Policy: The page’s settings blocked the loading of a resource at inline (“style-src”)."

The 'style' property should be set as an object, not as a string, according to CSP rules.

Back-ported from Blockly Games.

"Content-Security-Policy: The page’s settings blocked the loading of a resource at inline (“style-src”)."

The 'style' property should be set as an object, not as a string, according to CSP rules.

Back-ported from Blockly Games.
@NeilFraser NeilFraser requested a review from a team as a code owner September 19, 2023 09:26
@NeilFraser NeilFraser changed the title Setting style property to make CSP less grumpy. chore: Setting style property to make CSP less grumpy. Sep 19, 2023
@conventional-commit-lint-gcf
Copy link
Copy Markdown

conventional-commit-lint-gcf Bot commented Sep 19, 2023

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@github-actions github-actions Bot added the PR: chore General chores (dependencies, typos, etc) label Sep 19, 2023
Comment thread core/renderers/zelos/marker_svg.ts Outdated
},
this.markerSvg_,
);
this.markerCircle.style.display = 'none';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you also call hide here for consistency?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good call.

@NeilFraser NeilFraser merged commit 61cdbb6 into develop Sep 19, 2023
@NeilFraser NeilFraser deleted the fraser-style branch September 19, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: chore General chores (dependencies, typos, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants