If you'd prefer, you could just have each Assembly grab a serial number from the state when it is created
That would simplify the code a little, and it might have a slight advantage over…
We've decided to leave the element list update bug (#20) in this PR, and fix it in a subsequent PR.
I think best to fix in this PR, on the principle of not merging known buggy states of the code.
Great—I'll try to think about this before our meeting, and we can talk about it if it turns…
Commenting the example assembly chooser reminded me that the branch to be pulled has one outstanding bug: the element list doesn't always update properly when you choose a different assembly. I'd…
Either 'column' or 'columnIndex' would be better.
I've switched to column_index (in commit 7d6a394). To reduce clutter, I've kept index in the parts of Assembly::realize where the…
Yup. Stuff like "hello".to_string() looked even weirder to me at first, but the Rust book authors seem to consider it the most idiomatic way of going in the other direction...
Done (in commit 0c8022d). I've named the color variables according to their roles, with cues about the intended color limited to brightness descriptions. There are comments describing the colors…
Done (in commit 2b083be). Would it be helpful to think about what a better solution might look like? I started using these temporary emoji icons precisely because it felt too early to think about…
I think I recall you saying—maybe during a meeting—that you'd like Element::index to be renamed to something more descriptive, like col_index or config_col. If you do want that, let me…
Done! I've expanded "cst" to "constraint," "elt" to "element," and "rep" to "representation" in the HTML element classes (commits 882286c and 3f3c173).