Skip to content

Conversation

@maltheism
Copy link
Member

Brief summary of changes

Updated the Panel component to handle views if given. Also refactored the code to be a react functional component instead of a class. Used in PR: #8261

Testing instructions (if applicable)

  1. checkout pr
  2. make clean && make dev
  3. visit modules that use the Panel component and see if anything broke.

@maltheism maltheism added Category: Feature PR or issue that aims to introduce a new feature Category: Cleanup PR or issue introducing/requiring at least one clean-up operation Language: Javascript PR or issue that update Javascript code labels Dec 20, 2022
@maltheism maltheism requested review from driusan and laemtl December 20, 2022 21:53
@CamilleBeau
Copy link
Collaborator

The arrows to collapse the panel are not rendering correctly:

image

Things are looking good other than this!

@CamilleBeau CamilleBeau added the State: Needs work PR awaiting additional work by the author to proceed label Feb 27, 2023
@laemtl laemtl added this to the 25.0.0 milestone Feb 28, 2023
@laemtl laemtl removed the State: Needs work PR awaiting additional work by the author to proceed label Mar 16, 2023
@laemtl laemtl force-pushed the react_panel_multiple_views branch from e100df3 to 4a3fcce Compare March 16, 2023 16:10
@laemtl laemtl force-pushed the react_panel_multiple_views branch from 4a3fcce to ffad66a Compare March 16, 2023 16:50
@laemtl laemtl force-pushed the react_panel_multiple_views branch from ffad66a to 34e711a Compare March 16, 2023 16:53
@laemtl
Copy link
Contributor

laemtl commented Mar 16, 2023

Ready. I also rebased the PR to get rid of the unrelevant commits.
-- Update: I need to fix an integration test.

@laemtl laemtl added the State: Needs work PR awaiting additional work by the author to proceed label Mar 16, 2023
@laemtl laemtl removed the State: Needs work PR awaiting additional work by the author to proceed label Mar 16, 2023
@laemtl
Copy link
Contributor

laemtl commented Mar 16, 2023

Ready

@laemtl
Copy link
Contributor

laemtl commented Mar 20, 2023

@CamilleBeau Can you retest and approve if this fixes your issue?

@CamilleBeau
Copy link
Collaborator

@CamilleBeau Can you retest and approve if this fixes your issue?

Approved! It's fixed now. Thanks Laetitia!

@driusan driusan merged commit 22fc463 into aces:main Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Cleanup PR or issue introducing/requiring at least one clean-up operation Category: Feature PR or issue that aims to introduce a new feature Language: Javascript PR or issue that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants