Skip to content

Commit

Permalink
fix(ui): Platform on Edge should be normalized to 'edge' #17039
Browse files Browse the repository at this point in the history
  • Loading branch information
rstoenescu committed Mar 29, 2024
1 parent 8a1157e commit d937e63
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions ui/src/plugins/platform/Platform.js
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ function getPlatform (UA) {
delete browser[ 'windows phone' ]
}

if (browser.edga || browser.edgios) {
if (browser.edga || browser.edgios || browser.edg) {
browser.edge = true
matched.browser = 'edge'
}
Expand Down Expand Up @@ -154,14 +154,8 @@ function getPlatform (UA) {
browser.webkit = true
}

// TODO: (Qv3) rename the terms 'edge' to 'edge legacy' (or remove it) then 'edge chromium' to 'edge' to match with the known up-to-date terms
// Microsoft Edge is the new Chromium-based browser. Microsoft Edge Legacy is the old EdgeHTML-based browser (EOL: March 9, 2021).
if (browser.edg) {
matched.browser = 'edgechromium'
browser.edgeChromium = true
}
// Opera 15+ are identified as opr
else if (browser.opr) {
if (browser.opr) {
matched.browser = 'opera'
browser.opera = true
}
Expand Down

0 comments on commit d937e63

Please sign in to comment.