glen/dyna3
1
0
Fork 0
Code Issues 11 Pull Requests 1 Actions Projects Releases 2 Wiki Activity

Clean up the outline view #19

Merged
glen merged 23 commits from outline-cleanup_on_main into main 2024-11-15 03:32:48 +00:00
Conversation 9 Commits 23 Files Changed 7 +5 -1
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2b083be998 - Show all commits

4
app-proto/main.css
View File

@ -47,6 +47,10 @@ body {
} }
/* KLUDGE */ /* KLUDGE */
/*
for convenience, we're using emoji as temporary icons for some buttons. these
buttons need to be displayed in an emoji font
*/
#add-remove > button.emoji { #add-remove > button.emoji {
font-family: 'Noto Emoji', sans-serif; font-family: 'Noto Emoji', sans-serif;
} }

2
app-proto/src/add_remove.rs
View File

@ -173,7 +173,7 @@ pub fn AddRemove() -> View {
} }
) { "+" } ) { "+" }
button( button(
class="emoji", /* KLUDGE */ class="emoji", /* KLUDGE */ // for convenience, we're using an emoji as a temporary icon for this button
disabled={ disabled={
glen marked this conversation as resolved
glen commented 2024-11-13 17:54:02 +00:00
Review
Copy Link

Please elaborate all instances of KLUDGE

Please elaborate all instances of KLUDGE
👍 1
Vectornaut commented 2024-11-14 08:22:09 +00:00
Review
Copy Link

Done (in commit 2b083be).

Done (in commit 2b083be).
let state = use_context::<AppState>(); let state = use_context::<AppState>();
state.selection.with(|sel| sel.len() != 2) state.selection.with(|sel| sel.len() != 2)