Points #82

Merged
glen merged 16 commits from Vectornaut/dyna3:points into main 2025-05-01 19:25:14 +00:00
2 changed files with 11 additions and 3 deletions
Showing only changes of commit 07a415843d - Show all commits

View file

@ -42,9 +42,9 @@ body {
}
#add-remove > button {
width: 32px;
/*width: 32px;*/
height: 32px;
font-size: large;
/*font-size: large;*/
}
glen marked this conversation as resolved Outdated

Sorry I am on a mobile device and so will have to make individual comments instead of a single review, polluting your inbox. Anyhow, why are we leaving this css block in place now that it is empty? It is easy to restore in the future if we need it again. Not a fan of "dead code".

Sorry I am on a mobile device and so will have to make individual comments instead of a single review, polluting your inbox. Anyhow, why are we leaving this css block in place now that it is empty? It is easy to restore in the future if we need it again. Not a fan of "dead code".

Whoops, I forgot to remove those before committing. Removed in commit 35689e3.

Whoops, I forgot to remove those before committing. Removed in commit 35689e3.
/* KLUDGE */
@ -53,7 +53,9 @@ body {
buttons need to be displayed in an emoji font
*/
#add-remove > button.emoji {
width: 32px;
font-family: 'Noto Emoji', sans-serif;
font-size: large;
}
/* outline */

View file

@ -214,7 +214,13 @@ pub fn AddRemove() -> View {
let state = use_context::<AppState>();
state.assembly.insert_element_default::<Sphere>();
}
) { "+" }
) { "Add sphere" }
button(
on:click=|_| {
let state = use_context::<AppState>();
state.assembly.insert_element_default::<Point>();
}
) { "Add point" }
button(
class="emoji", /* KLUDGE */ // for convenience, we're using an emoji as a temporary icon for this button
disabled={