Skip to content

refactor: use new form on RangesPickingOptionPanel#4255

Open
Sebastien-Ahkrin wants to merge 2 commits into
mainfrom
4175-refactor-useform-on-rangespickingoptionpanel
Open

refactor: use new form on RangesPickingOptionPanel#4255
Sebastien-Ahkrin wants to merge 2 commits into
mainfrom
4175-refactor-useform-on-rangespickingoptionpanel

Conversation

@Sebastien-Ahkrin

Copy link
Copy Markdown
Collaborator

closes: #4175

@Sebastien-Ahkrin Sebastien-Ahkrin linked an issue Jul 8, 2026 that may be closed by this pull request
@Sebastien-Ahkrin Sebastien-Ahkrin requested a review from targos July 8, 2026 12:55
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 8, 2026

Copy link
Copy Markdown

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: b5807a9
Status: ✅  Deploy successful!
Preview URL: https://bad6e773.nmrium.pages.dev
Branch Preview URL: https://4175-refactor-useform-on-ran.nmrium.pages.dev

View logs

@targos targos requested review from tpoisseau and removed request for targos July 8, 2026 12:56
@targos

targos commented Jul 8, 2026

Copy link
Copy Markdown
Member

Please ask @tpoisseau for review on these tasks.

Comment thread src/component/header/RangesPickingOptionPanel.tsx Outdated
Comment thread src/component/header/RangesPickingOptionPanel.tsx Outdated
Comment on lines +24 to +29
const AppFormContainer = styled(AppForm)`
display: flex;
flex-direction: row;
gap: 10px;
align-items: center;
`;

@tpoisseau tpoisseau Jul 8, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You could style Checkbox and FormGroup here too.

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.

Refactor useForm on RangesPickingOptionPanel

3 participants