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

first steps of project industrialisation #23

Merged
merged 29 commits into from
Apr 8, 2024
Merged

first steps of project industrialisation #23

merged 29 commits into from
Apr 8, 2024

Conversation

kmer2016
Copy link
Contributor

@kmer2016 kmer2016 commented Apr 8, 2024

  • Add some component of design system
  • set project architecture

thibautsailly and others added 29 commits January 29, 2024 17:04
- create-react-app ui-speedspacechart
- add taillwindcss using PostCSS
- copy /ui-core/styles/main.css to replace /ui-speedspacechart/index.css
- import main.css in index.tsx, remove import from App.tsx
- edit tailwind.config.js
- add IBMPlex fonts to /public
- add storybook, clean /stories
- clean unnecessary files
- create speedSpaceChart story
- create useCanvas custom hook
- create:
  - SpeedSpaceChart.tsx
  - FrontInterractivityLayer.tsx
  - GridLayer.tsx
  - CurveLayer.tsx
- add draw function in drawElements.ts
- copy simulation consts and types from osrd
- copy sampleData from osrd
- add ticks on axis
- add d3 zoom function for zoom and reset
- replace speed props with full store props
Details:
- Drafts WarpedMap, based on existing code from OSRD, but without all
  the business specific code
- Drafts first story (but cannot run it because of missing data)

TODO:
- Add missing features:
  - Allow syncing bbox
  - Allow refining data (ie. reloading OSRD entities)
  - Allow adding custom buttons (to set bbox sync mode)
- Add missing data to base story
Details:
- Adds sample paths and OSM styles
- Adds an optional timeout to DataLoader
- Fixes various minor issues to get a proper first rendered map

TODO:
- Add missing features, detailed in previous commit
Details:
- Makes every setting from getWarping overridable
- Adds inputs for all warping settings in algorithm showcase stories
Details:
- Intersects polygons with zone instead of clipping them (this makes
  large areas of water for instance being properly drawn around the
  grid)
- Fixes various issues that caused properties to not be properly set to
  the warped data
- Fixes TransformedDataMap filters to take source-layers into account
- monorepo project with sub-packages which allow us to share dep,
  scripts, ...
- rollup configuration (remove commonjs, babel, and adding postcss)
- package.json with export
- remove CRA trace
- adding a stroybook package that read stories across project
- adding lerna for release
* ui-core: implement anatomy input

ui-core: use css instead of full tailwind class

ui-core: handle small input

ui-core: readme v1

ui-core: implement disabled & readonly

ui-core: fix height of component, use bowshadow instead of border

ui-core: fix small input

ui-core: fix style on focus via keyboard

ui-core: use box shadow fof leading/trailing

iu-core: fix blur

ui-core: implement token input

fix border using box shadow

ui-core: add form validation

refacto style

refacto. use scss power to handle small version of input more clearly

* ui-core: wrap token input

* ui-core: password input

* Fix story book and styles

* ui-core: export Input, TokenInput and PasswordInput

---------

Co-authored-by: Benoit Simard <[email protected]>
…-prettier-conf

osrd-ui: normalize eslint and prettier conf
…config-for-storybook-hot-reload

ui-core: optimize rollup config for  storybook hotreload
* osrd-ui: build and check code

* fix prettier error

* osrd-ui: ignore prettier error from component generated by ui-icons

* osrd-ui: enable hot reload only when we are in development
@kmer2016 kmer2016 changed the title Dev first steps of project industrialisation Apr 8, 2024
@kmer2016 kmer2016 merged commit ec7848f into main Apr 8, 2024
2 of 3 checks passed
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.

8 participants