Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

webapp: fix creation form #850

Draft
wants to merge 26 commits into
base: develop
Choose a base branch
from
Draft

Conversation

tharvik
Copy link
Collaborator

@tharvik tharvik commented Dec 19, 2024

see #690 for context

  • webapp
    • redo form
      • split into more categories
      • make it nicer (no more loooong input field, grid layout on bigger screen)
      • greatly simplify fields implementation (no more type specific container, no dynamic form)
    • rework the internal layout of many pages to better separate concerns
      • avoid some forced sizing
      • have common cards-gap CSS class instead of inconsistent gap
    • nicer data example layout
    • testing
      • fix unconsistent temporary server URL
      • spin test server on another port to avoid having to kill dev-server to launch tests
  • discojs
    • mv Task.TrainingInformation.dataType to Task.dataType
      • allows Task.DisplayInformation to be type dependant
    • redo most of Task.DisplayInformation

@tharvik tharvik force-pushed the 690-fix-creation-form-tharvik branch from 59c96ac to 36997ea Compare December 21, 2024 00:54
@tharvik tharvik force-pushed the 690-fix-creation-form-tharvik branch 11 times, most recently from 76f356c to f08c180 Compare January 10, 2025 11:47
@tharvik tharvik force-pushed the 690-fix-creation-form-tharvik branch from 4d33511 to 8189e7f Compare January 16, 2025 12:30
@tharvik tharvik force-pushed the 690-fix-creation-form-tharvik branch from 8189e7f to 46f5830 Compare January 21, 2025 17:23
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