Skip to content

Commit b73c1a5

Browse files
DYN-8121 SplashScreen Disable Close Button (#81)
* DYN-8121 SplashScreen Disable Close Button Remove all the code related to Close Button in SplashScreen * Fixing playwright/test package Updating the playwright/test package to 1.49.0 * Fixing CodeCoverage report bumping : zgosalvez/github-actions-report-lcov action to v2 just to see if now it passes. * Fixing CodeCoverage report github action bumping : zgosalvez/github-actions-report-lcov action to v3 just to see if now it passes. * Fixing CodeCoverage report adding more code for the github action zgosalvez/github-actions-report-lcov@v3 * Fixing CodeCoverage report removing lcov data * Revert "Fixing CodeCoverage report" This reverts commit 5b03d4b. * Fixing CodeCoverage report bumping up version * Fixing CodeCoverage report - update action versions - add step to setup lcov * Bumping up version bumping up version from 1.0.26 to 1.0.27
1 parent da98f0b commit b73c1a5

5 files changed

Lines changed: 57 additions & 73 deletions

File tree

.github/workflows/build.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ jobs:
1818
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/
1919

2020
steps:
21-
- uses: actions/checkout@v3
21+
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
2222
- name: Use Node.js ${{ matrix.node-version }}
23-
uses: actions/setup-node@v3
23+
uses: actions/setup-node@39370e3970a6d050c480ffad4ff0ed4d3fdee5af # v4.1.0
2424
with:
2525
node-version: ${{ matrix.node-version }}
2626
cache: "npm"
@@ -34,8 +34,12 @@ jobs:
3434
run: npx playwright install chromium --with-deps
3535
- name: E2E tests
3636
run: npm run test:e2e
37+
- name: Setup LCOV
38+
uses: hrishikesh-kadam/setup-lcov@6c1aa0cc9e1c02f9f58f01ac599f1064ccc83470 # v1.1.0
39+
with:
40+
ref: v1.16
3741
- name: Code Coverage Report
38-
uses: zgosalvez/github-actions-report-lcov@v1
42+
uses: zgosalvez/github-actions-report-lcov@5989987f8058a03137e90bc16f9c0baaac5e069a # v4.1.22
3943
with:
4044
coverage-files: coverage/lcov.info
4145
# minimum-coverage: 50

package-lock.json

Lines changed: 48 additions & 22 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@dynamods/splash-screen",
3-
"version": "1.0.26",
3+
"version": "1.0.27",
44
"description": "Splash Screen maintained by Dynamo Team@Autodesk",
55
"author": "Autodesk Inc.",
66
"license": "MIT",
@@ -56,7 +56,7 @@
5656
]
5757
},
5858
"devDependencies": {
59-
"@playwright/test": "^1.27.1",
59+
"@playwright/test": "^1.49.0",
6060
"@testing-library/react": "^13.4.0",
6161
"html-webpack-plugin": "^4.5.2",
6262
"webpack": "^5.74.0",

src/App.css

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -19,35 +19,4 @@
1919
.screenBackground {
2020
height: 412px;
2121
width: 394px;
22-
}
23-
24-
.close {
25-
position: absolute;
26-
cursor: pointer;
27-
right: 32px;
28-
top: 15px;
29-
width: 16px;
30-
height: 16px;
31-
opacity: 0.3;
32-
}
33-
34-
.close:hover {
35-
opacity: 1;
36-
}
37-
38-
.close:before, .close:after {
39-
position: absolute;
40-
left: 25px;
41-
content: ' ';
42-
height: 16px;
43-
width: 2px;
44-
background-color: #333;
45-
}
46-
47-
.close:before {
48-
transform: rotate(45deg);
49-
}
50-
51-
.close:after {
52-
transform: rotate(-45deg);
5322
}

src/App.js

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ class App extends React.Component {
2626
window.setLoadingDone = this.setLoadingDone.bind(this);
2727
window.setSignInStatus = this.setSignInStatus.bind(this);
2828
this.handleCheckedChange = this.handleCheckedChange.bind(this);
29-
this.closeDynamo = this.closeDynamo.bind(this);
3029
}
3130

3231
handleCheckedChange = (checked) => {
@@ -45,14 +44,6 @@ class App extends React.Component {
4544
//TODO : As alternative we can receive the event from the Childs like the Static component
4645
}
4746

48-
handleKeyDown = (e) => {
49-
if (e.key === 'Escape') {
50-
if (this.state.loadingDone) {
51-
this.closeDynamo();
52-
}
53-
}
54-
};
55-
5647
render() {
5748
return (
5849
<Container fluid>
@@ -132,12 +123,6 @@ class App extends React.Component {
132123
loadingDone: true
133124
});
134125
};
135-
136-
closeDynamo() {
137-
if (chrome.webview !== undefined) {
138-
chrome.webview.hostObjects.scriptObject.CloseWindowPreserve(this.state.isChecked);
139-
}
140-
}
141126
}
142127

143128
export default App;

0 commit comments

Comments
 (0)