Skip to content

Better & non-distracting live preview highlighting #2706

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.

Sign up for GitHub

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

Open
devvaannsh wants to merge 31 commits into
base: main
Choose a base branch
from
+697 −432

fix: element doesn’t get refreshed until mode is changed

ee9b46e
Select commit
Loading
Failed to load commit list.
Open

Better & non-distracting live preview highlighting #2706

fix: element doesn’t get refreshed until mode is changed
ee9b46e
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 13, 2026 in 47s

Quality Gate passed

Issues
45 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 1302 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected a `for-of` loop instead of a `for` loop with this simple iteration. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzE7nidAcnzKZukzkhO&open=AZzE7nidAcnzKZukzkhO&pullRequest=2706

Check warning on line 1234 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZypeqx5sZaVqOVfxMy_&open=AZypeqx5sZaVqOVfxMy_&pullRequest=2706

Check warning on line 542 in src/LiveDevelopment/BrowserScripts/LiveDevProtocolRemote.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZy1Ca-AvoVWQ2TYI4mh&open=AZy1Ca-AvoVWQ2TYI4mh&pullRequest=2706

Check warning on line 790 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'redrawUIBoxes' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzb-Ezjd1orA-MjPBjK&open=AZzb-Ezjd1orA-MjPBjK&pullRequest=2706

Check warning on line 542 in src/LiveDevelopment/BrowserScripts/LiveDevProtocolRemote.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZy1Ca-AvoVWQ2TYI4mi&open=AZy1Ca-AvoVWQ2TYI4mi&pullRequest=2706

Check warning on line 294 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gf&open=AZyJ-zA9IPT-dly1h1Gf&pullRequest=2706

Check warning on line 751 in src/LiveDevelopment/LiveDevMultiBrowser.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzmYlIflpXvOsy3Axey&open=AZzmYlIflpXvOsy3Axey&pullRequest=2706

Check warning on line 387 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1G0&open=AZyJ-zA9IPT-dly1h1G0&pullRequest=2706

Check warning on line 666 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected a `for-of` loop instead of a `for` loop with this simple iteration. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1G2&open=AZyJ-zA9IPT-dly1h1G2&pullRequest=2706

Check failure on line 168 in src/LiveDevelopment/MultiBrowserImpl/documents/LiveHTMLDocument.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected var, use let or const instead. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzODZGLl-vY2hRLbAcz&open=AZzODZGLl-vY2hRLbAcz&pullRequest=2706

Check warning on line 329 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gj&open=AZyJ-zA9IPT-dly1h1Gj&pullRequest=2706

Check warning on line 1420 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyVpo6KHrekD60xYUnj&open=AZyVpo6KHrekD60xYUnj&pullRequest=2706

Check warning on line 318 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gg&open=AZyJ-zA9IPT-dly1h1Gg&pullRequest=2706

Check warning on line 1118 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzODZMFl-vY2hRLbAc2&open=AZzODZMFl-vY2hRLbAc2&pullRequest=2706

Check warning on line 116 in src/extensionsIntegrated/Phoenix-live-preview/live-preview.css

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Text does not meet the minimal contrast requirement with its background. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZymGiDoeQUfltL-kPkr&open=AZymGiDoeQUfltL-kPkr&pullRequest=2706

Check warning on line 321 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use the `this` argument in `Array#forEach()`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gh&open=AZyJ-zA9IPT-dly1h1Gh&pullRequest=2706

Check warning on line 1275 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_getTreePath' to the outer scope. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzE7nidAcnzKZukzkhM&open=AZzE7nidAcnzKZukzkhM&pullRequest=2706

Check failure on line 170 in src/LiveDevelopment/MultiBrowserImpl/documents/LiveHTMLDocument.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected var, use let or const instead. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzODZGLl-vY2hRLbAc1&open=AZzODZGLl-vY2hRLbAc1&pullRequest=2706

Check warning on line 414 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1G1&open=AZyJ-zA9IPT-dly1h1G1&pullRequest=2706

Check warning on line 1278 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzE7nidAcnzKZukzkhN&open=AZzE7nidAcnzKZukzkhN&pullRequest=2706

Check warning on line 274 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Ge&open=AZyJ-zA9IPT-dly1h1Ge&pullRequest=2706

Check failure on line 545 in src/LiveDevelopment/BrowserScripts/LiveDevProtocolRemote.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected var, use let or const instead. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZy1Ca-AvoVWQ2TYI4mj&open=AZy1Ca-AvoVWQ2TYI4mj&pullRequest=2706

Check warning on line 339 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseFloat` over `parseFloat`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gp&open=AZyJ-zA9IPT-dly1h1Gp&pullRequest=2706

Check warning on line 386 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `Number.parseFloat` over `parseFloat`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZzb-Ezjd1orA-MjPBjI&open=AZzb-Ezjd1orA-MjPBjI&pullRequest=2706

Check warning on line 328 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZyJ-zA9IPT-dly1h1Gi&open=AZyJ-zA9IPT-dly1h1Gi&pullRequest=2706
View more details on SonarQubeCloud