-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Sandcastle Reborn #12574
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.
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
Draft
jjspace
wants to merge
21
commits into
sandcastle-v2
Choose a base branch
from
sandcastle-reborn
base: sandcastle-v2
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+3,987
−10
Draft
Sandcastle Reborn #12574
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
eb55c7e
create sandcastle package
jjspace 793ba88
elevate eslint config, run prettier
jjspace 8a73405
import hackathon code
jjspace c05ed50
adjusting paths for built and local dev
jjspace 4a2082a
readme and build commands
jjspace 1d8cd04
fix ci
jjspace 1522aa7
set up itwin-ui, fix types route
jjspace f775608
build for ci
jjspace c3e3f11
Config setups, including CI
ggetz db51183
remove extra files
jjspace 414fc15
Merge remote-tracking branch 'origin/main' into sandcastle-reborn
jjspace 463c6b3
test html build
jjspace 5c02885
adjust paths
jjspace 93955b7
adjust paths again
jjspace f714fa2
switch to custom plugin and vite define
jjspace ebafd4d
ci test
jjspace ab625d8
remove static bucket.html
jjspace f574e97
adjust configs slightly
jjspace fc38e02
streamline vite config with custom plugin
jjspace 8fa32a4
fix types url
jjspace 2ac1ed3
nest bucket.html like in current sandcastle
jjspace File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
/node_modules | ||
packages/sandcastle/node_modules | ||
/ThirdParty | ||
/Tools/** | ||
|
||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
# Logs | ||
logs | ||
*.log | ||
npm-debug.log* | ||
yarn-debug.log* | ||
yarn-error.log* | ||
pnpm-debug.log* | ||
lerna-debug.log* | ||
|
||
node_modules | ||
dist | ||
dist-ssr | ||
*.local | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,61 @@ | ||
# CesiumJS Sandcastle | ||
|
||
This package is the application for Sandcastle. | ||
|
||
## Running/Building | ||
|
||
- `npm run dev`: run the development server | ||
- `npm run build`: build locally with vite defaults | ||
- `npm run preview`: run the production build locally. <!--TODO: I'm not sure if we actually need this for our purposes --> | ||
- `npm run build-app`: build to static files in `/Apps/Sandcastle2` for hosting/access from the root cesium dev server | ||
- `npm run build-ci`: build to static files in `/Apps/Sandcastle2` and configure paths as needed for CI deployment | ||
|
||
Linting and style is managed under the project root's scripts. | ||
|
||
## Expanding the ESLint configuration | ||
|
||
<!-- TODO: this section was auto-generated, should figure out if we want these suggestions then remove this --> | ||
|
||
If you are developing a production application, we recommend updating the configuration to enable type-aware lint rules: | ||
|
||
```js | ||
export default tseslint.config({ | ||
extends: [ | ||
// Remove ...tseslint.configs.recommended and replace with this | ||
...tseslint.configs.recommendedTypeChecked, | ||
// Alternatively, use this for stricter rules | ||
...tseslint.configs.strictTypeChecked, | ||
// Optionally, add this for stylistic rules | ||
...tseslint.configs.stylisticTypeChecked, | ||
], | ||
languageOptions: { | ||
// other options... | ||
parserOptions: { | ||
project: ["./tsconfig.node.json", "./tsconfig.app.json"], | ||
tsconfigRootDir: import.meta.dirname, | ||
}, | ||
}, | ||
}); | ||
``` | ||
|
||
You can also install [eslint-plugin-react-x](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-x) and [eslint-plugin-react-dom](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-dom) for React-specific lint rules: | ||
|
||
```js | ||
// eslint.config.js | ||
import reactX from "eslint-plugin-react-x"; | ||
import reactDom from "eslint-plugin-react-dom"; | ||
|
||
export default tseslint.config({ | ||
plugins: { | ||
// Add the react-x and react-dom plugins | ||
"react-x": reactX, | ||
"react-dom": reactDom, | ||
}, | ||
rules: { | ||
// other rules... | ||
// Enable its recommended typescript rules | ||
...reactX.configs["recommended-typescript"].rules, | ||
...reactDom.configs.recommended.rules, | ||
}, | ||
}); | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
<!doctype html> | ||
<html lang="en"> | ||
<head> | ||
<meta charset="UTF-8" /> | ||
<link rel="icon" type="image/svg+xml" href="/favicon.ico" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>Sandcastle Reborn</title> | ||
<style> | ||
/* Load fonts for itwin-ui */ | ||
@font-face { | ||
font-family: InterVariable; | ||
font-style: normal; | ||
font-weight: 100 900; | ||
font-display: swap; | ||
src: url("/fonts/InterVariable.woff2") format("woff2"); | ||
} | ||
|
||
@font-face { | ||
font-family: InterVariable; | ||
font-style: italic; | ||
font-weight: 100 900; | ||
font-display: swap; | ||
src: url("/fonts/InterVariable-Italic.woff2") format("woff2"); | ||
} | ||
</style> | ||
</head> | ||
<body> | ||
<div id="app-container"></div> | ||
<script type="module" src="/src/main.tsx"></script> | ||
</body> | ||
</html> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
{ | ||
"name": "@cesium/sandcastle", | ||
"private": true, | ||
"version": "0.0.1", | ||
"type": "module", | ||
"scripts": { | ||
"dev": "vite --config vite.config.dev.ts", | ||
"build": "tsc -b && vite build --config vite.config.dev.ts", | ||
"preview": "vite preview --config vite.config.dev.ts", | ||
"build-app": "tsc -b && vite build --config vite.config.app.ts", | ||
"build-ci": "tsc -b && vite build --config vite.config.ci.ts" | ||
}, | ||
"dependencies": { | ||
"@itwin/itwinui-react": "^5.0.0-alpha.14", | ||
"@monaco-editor/react": "^4.7.0", | ||
"monaco-editor": "^0.52.2", | ||
"pako": "^2.1.0", | ||
"react": "^19.0.0", | ||
"react-dom": "^19.0.0" | ||
}, | ||
"devDependencies": { | ||
"@types/pako": "^2.0.3", | ||
"@types/react": "^19.0.10", | ||
"@types/react-dom": "^19.0.4", | ||
"@vitejs/plugin-react": "^4.3.4", | ||
"globals": "^15.15.0", | ||
"typescript": "~5.7.2", | ||
"vite": "^6.2.0", | ||
"vite-plugin-static-copy": "^2.3.1" | ||
} | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to target workspace which are dependencies to avoid the manual filtering.