Clarify structure operations

[edan-bainglass]

This PR aims to emphasize the distinction between visualizing the super cell and actually transforming it. It does so by (i) wrapping the viewer in its own accordion titled "View structure", and (ii) adding a comment below the super cell control in the appearance tab of the viewer clarifying usage.

Closes #667

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.41%. Comparing base (0aaf8fe) to head (3353ddb).
Report is 7 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #666   +/-   ##
=======================================
  Coverage   83.41%   83.41%           
=======================================
  Files          17       17           
  Lines        3581     3581           
=======================================
  Hits         2987     2987           
  Misses        594      594           

Flag	Coverage Δ
python-3.11	83.41% <100.00%> (ø)
python-3.9	83.44% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[AndresOrtegaGuerrero]

LGTM

[AndresOrtegaGuerrero]

I wonder thy now more tests are failing

[edan-bainglass]

I wonder thy now more tests are failing

Hmm, that's very odd indeed. Not sure what the COD database has to do with this PR. Will check before merging.

[danielhollas]

That particular test is flaky, reported in #649. It needs to be refactored because it accesses the live COD database. Instead the DB access needs to be mocked out.

[edan-bainglass]

That particular test is flaky, reported in #649. It needs to be refactored because it accesses the live COD database. Instead the DB access needs to be mocked out.

Thanks @danielhollas. Forgot I even opened that one. Too many issues/PRs! Indeed, a simple re-run was sufficient.