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

Refactor editor panels #245

Merged
merged 4 commits into from
Nov 28, 2023
Merged

Refactor editor panels #245

merged 4 commits into from
Nov 28, 2023

Conversation

formsdev
Copy link
Collaborator

No description provided.

@formsdev formsdev requested a review from JhumanJ November 23, 2023 07:21
Copy link
Owner

@JhumanJ JhumanJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that we have a component for the panel we can refactor and remove all the isCollapseOpen from subcomponents and move them in EditorPanel to simplify them. The color switch of icon should be handled in EditorOptionsPanel to remove duplicated code

<editor-options-panel name="About Submissions" v-model="isCollapseOpen">
<template #icon>
<svg
class="h-5 w-5 inline mr-2 -mt-0.5 transition-colors" :class="{'text-blue-600':isCollapseOpen, 'text-gray-500':!isCollapseOpen}"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here all classes excepct the width/height must be removed. You can put the rest if needed in EditorOptionsPanel, so that we won't have to do the same thing for the blue color in all components. you can also remove isCollapseOpen since we won't need it anymore in this component after this change

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@formsdev formsdev requested a review from JhumanJ November 24, 2023 03:58
@JhumanJ JhumanJ merged commit 730bdd1 into main Nov 28, 2023
0 of 2 checks passed
@JhumanJ JhumanJ deleted the c3303-refactor-editor-panels branch November 28, 2023 10:24
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.

2 participants