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 28 commits into
base: main
Choose a base branch
from
+697 −416

feat: add styler row drill in strings

4854814
Select commit
Loading
Failed to load commit list.
Open

Better & non-distracting live preview highlighting #2706

feat: add styler row drill in strings
4854814
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 9, 2026 in 45s

Quality Gate passed

Issues
51 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 321 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-dly1h1Go&open=AZyJ-zA9IPT-dly1h1Go&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 `Number.parseFloat` over `parseFloat`. 

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

Check warning on line 324 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-dly1h1Gr&open=AZyJ-zA9IPT-dly1h1Gr&pullRequest=2706

Check warning on line 807 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=AZypeqx5sZaVqOVfxMy9&open=AZypeqx5sZaVqOVfxMy9&pullRequest=2706

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code. 

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

Check failure on line 421 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed. 

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

Check warning on line 323 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-dly1h1Gq&open=AZyJ-zA9IPT-dly1h1Gq&pullRequest=2706

Check failure on line 1026 in src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 70 to the 15 allowed. 

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

Check warning on line 1135 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 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 325 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-dly1h1Gs&open=AZyJ-zA9IPT-dly1h1Gs&pullRequest=2706

Check warning on line 353 in src/extensionsIntegrated/Phoenix-live-preview/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times. 

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

Check warning on line 1313 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 1333 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=AZzE7nidAcnzKZukzkhP&open=AZzE7nidAcnzKZukzkhP&pullRequest=2706

Check warning on line 361 in src/extensionsIntegrated/Phoenix-live-preview/main.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not call `Array#push()` multiple times. 

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

Check warning on line 322 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 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 301 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 320 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-dly1h1Gn&open=AZyJ-zA9IPT-dly1h1Gn&pullRequest=2706

Check warning on line 332 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-dly1h1Gx&open=AZyJ-zA9IPT-dly1h1Gx&pullRequest=2706

Check warning on line 286 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 327 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-dly1h1Gu&open=AZyJ-zA9IPT-dly1h1Gu&pullRequest=2706

Check warning on line 304 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 failure on line 545 in src/LiveDevelopment/BrowserScripts/LiveDevProtocolRemote.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`. 

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

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

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function 'Highlight' to the outer scope. 

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