change visualization parameters #10
Conversation
|
✔️ Deploy Preview for radiotherapy-plans-visualization ready! 🔨 Explore the source changes: 91e9dbe 🔍 Inspect the deploy log: https://app.netlify.com/sites/radiotherapy-plans-visualization/deploys/615b4b25b843c00007485a3f 😎 Browse the preview: https://deploy-preview-10--radiotherapy-plans-visualization.netlify.app/ |
src/components/ThreeCanvas.tsx
Outdated
| } | ||
|
|
||
| export default function ThreeCanvas({ appBarHeight }: ThreeCanvasProps) { | ||
| export default function ThreeCanvas(props: ThreeCanvasProps) { |
There was a problem hiding this comment.
W js można odrazu robić destrukcję obiektów w argumentach funkcji.
Wtedy nie trzeba by było pisać props.[PROP_NAME], a można by było
ThreeCanvas({
appBarHeight,
colorIntensity,
...
}:ThreeCanvasProps) {
....There was a problem hiding this comment.
ok. Poprawione.
src/App.tsx
Outdated
| selectableRegions={["maxilla", "jawbone"]} | ||
| onRegionsChange={(regions) => { | ||
| setRegions(regions); | ||
| console.log(regions[0].transparency); |
There was a problem hiding this comment.
Racja. Usuniete ;)
src/App.tsx
Outdated
| onRegionsChange={(regions) => { | ||
| setRegions(regions); | ||
| console.log(regions[0].transparency); | ||
| }} |
There was a problem hiding this comment.
Można też
onRegionsChange={setRegions}
| colorIntensity={brightness / 100} | ||
| opacity={regions.length > 0 ? regions[0].transparency / 100 : 0.8} |
There was a problem hiding this comment.
Według mnie lepiej przekazywać brightness itransperency od razu w procentach, żeby w przypadku jeśli będziemy chcieli jeszcze gdzieś tego użyć nie musieliśmy znowu dzielić przez 100.
There was a problem hiding this comment.
Slider przechowuje wartości 0-100, więc chyba lepiej tak jak jest zostawic.
src/components/BasicSettings.tsx
Outdated
| export default function BasicSettings() { | ||
| interface BasicSettingsProps { | ||
| brightness: number; | ||
| onBrightnessChange: (event: unknown, newValue: number | number[]) => void; |
There was a problem hiding this comment.
Nie wiem, taka sygnatura jest w Sliderze z material-ui
src/components/BasicSettings.tsx
Outdated
| onCutChange: (event: unknown, newValue: number | number[]) => void; | ||
| } | ||
|
|
||
| export default function BasicSettings(props: BasicSettingsProps) { |
There was a problem hiding this comment.
Widzę, że taka nazwa już była. ale dopiero teraz rzuciła mi się w oczy.
Czy nie lepiej nazwać GlobalSettings?
src/components/Slider.tsx
Outdated
| title: string; | ||
| leftIcon: SVGProps<SVGElement>; | ||
| rightIcon: SVGProps<SVGElement>; | ||
| value: number; | ||
| onValueChange: (event: unknown, newValue: number | number[]) => void; | ||
| }; |
There was a problem hiding this comment.
Żeby za każdym razem nie dodawać nowych wartości do Slider'a można by było zrobić tak, żeby ten interfejs rozszerzał propsy które dostaje Slider z material-ui
| let plane = new Plane(new Vector3(0, 0, -1), props.startPoint - props.radius); | ||
| gl.clippingPlanes = [plane]; | ||
| gl.localClippingEnabled = true; | ||
| return <></>; |
There was a problem hiding this comment.
Nie. Wystarczy, by funkcja się uruchomiła.
No description provided.