Skip to content

correct delete-icon to icon-delete#66

Merged
jancborchardt merged 1 commit into
masterfrom
delete-icon-rename
Dec 30, 2014
Merged

correct delete-icon to icon-delete#66
jancborchardt merged 1 commit into
masterfrom
delete-icon-rename

Conversation

@jancborchardt

Copy link
Copy Markdown
Contributor

Please review @Raydiation

jancborchardt added a commit that referenced this pull request Dec 30, 2014
correct delete-icon to icon-delete
@jancborchardt jancborchardt merged commit b848331 into master Dec 30, 2014
@jancborchardt jancborchardt deleted the delete-icon-rename branch December 30, 2014 13:50
@jancborchardt

Copy link
Copy Markdown
Contributor Author

Merged cause this fixes the non-displaying delete icon.

phil-davis pushed a commit that referenced this pull request Jul 9, 2026
All alerts are transitive dev-scope deps, remediated via npm `overrides`
in js/package.json. Verified: `npm install` clean, `gulp` build green,
karma/jasmine suite 22/22 passing (postcss 7->8 and merge 1->2 are major
bumps — build and tests confirmed unaffected).

- js-yaml 3.14.2 -> 3.15.0 (#71, GHSA quadratic-complexity DoS, needs >= 3.15.0)
- lodash 4.17.23 -> 4.18.1 (#62 code injection via _.template, #63 prototype
  pollution in _.unset/_.omit, needs >= 4.18.0)
- merge 1.2.1 -> 2.1.1 (#14 prototype pollution, needs >= 2.1.1; major bump)
- minimatch 3.0.4/3.1.2 -> 3.1.5 (#54 ReDoS in matchOne(), needs >= 3.1.3)
- minimist (under optimist) 0.0.10 -> 0.2.4 (#30 critical + #9 prototype
  pollution, needs >= 0.2.4; per-parent override, root minimist already 1.2.8)
- postcss 7.0.39 -> 8.5.16 (#66 XSS via unescaped </style>, #35 line-return
  parsing, needs >= 8.5.10; major bump, no 7.x patch exists)

Note: `npm audit` still reports separate issues (brace-expansion, es5-ext,
micromatch, socket.io-parser, ws) that are NOT in the Dependabot alert set;
out of scope for this change.

Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
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.

1 participant