Skip to content

[Bug] LCT: Strange things with S #858

Open

Description

@smtdm

Describe the bug
S has been fixed (it now can have 6 bonds). But this breaks some rendering.
Maybe we need to redefine how valence electrons are used in the mod.

To Reproduce

  1. Take LCT with S
  2. Add some stuff
  3. ???
  4. Profit (or crash)

Expected behavior
S (and Se and Te) should be able to make 6 bonds, like in H2SO4.
When adding 4 H's next to S, the final 2 lone electrons need to be rendered somewhere (first screenshot)

Screenshots
afbeelding

An attempt to fix S:
afbeelding

Additional context

Activity

  1. removed their assignment
    on Feb 16, 2024
  2. pixar02 commented on Feb 17, 2024

    @pixar02
    Collaborator

    Is caused because you "replace" the item in the grid with the item in your cursor.
    So we need to make sure we can't "replace"/ "swap" items (grid <-> cursor)

  3. added a commit that references this issue on Feb 19, 2024
    0ea9edf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

  • Status
    📋 Backlog

Relationships

None yet

Development

No branches or pull requests

Issue actions