Skip to content

Generic Element analysis options#2908

Open
alizaib1217 wants to merge 8 commits intomainfrom
element_analysis_options
Open

Generic Element analysis options#2908
alizaib1217 wants to merge 8 commits intomainfrom
element_analysis_options

Conversation

@alizaib1217
Copy link
Contributor

@alizaib1217 alizaib1217 commented Jan 27, 2026

Description

Adds analysis options for generic elements to match the functionality available in Sample and other static elements, addressing
LabIMotion/labimotion#66.

Features Added

  • General Comment: Users can add a general description/comment to the analyses tab
  • Order Mode: Users can reorder analyses via drag-and-drop with a dedicated order mode toggle
  • Analyses from Upload: Users can create multiple analyses at once by uploading files or folders
  • Add to Report: User can now click checkbox of analysis to add it to report

Bug Fixes

  • Fixed analysis order not persisting after save and page reload

  • rather 1-story 1-commit than sub-atomic commits

  • commit title is meaningful => git history search

  • commit description is helpful => helps the reviewer to understand the changes

  • code is up-to-date with the latest developments of the target branch (rebased to it or whatever) => ⏩-merge for linear history is favoured

  • added code is linted

  • tests are passing (at least locally): we still have some random test failure on CI. thinking of asking spec/examples.txt to be commited

  • in case the changes are visible to the end-user,  video or screenshots should be added to the PR => helps with user testing

  • testing coverage improvement is improved.

  • CHANGELOG :  add a bullet point on top (optional: reference to github issue/PR )

  • parallele PR for documentation  on docusaurus  if the feature/fix is tagged for a release

@alizaib1217 alizaib1217 self-assigned this Jan 27, 2026
@alizaib1217 alizaib1217 changed the title Element analysis options Generic Element analysis options Jan 27, 2026
@alizaib1217 alizaib1217 requested review from phuang26 and removed request for phuang26 January 28, 2026 09:02
@alizaib1217 alizaib1217 marked this pull request as ready for review January 28, 2026 09:07
@alizaib1217 alizaib1217 force-pushed the element_analysis_options branch from 89d24bf to b67ee19 Compare February 4, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant