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

F t34968 add dp bulk edit header to storybook #615

Merged
merged 29 commits into from
Nov 3, 2023

Conversation

ahmad-demos
Copy link
Contributor

@ahmad-demos ahmad-demos commented Oct 25, 2023

https://yaits.demos-deutschland.de/T34968
demos-europe/demosplan-core#2245

  • WIP: Add DpBulkEditHeader to Storybook.
  • Add Tran-Key to DpBulkEditHeader component.

Screenshot 2023-11-01 at 13-04-28 Components _ BulkEditHeader - Default ⋅ Storybook
Screenshot 2023-11-01 at 13-04-39 Components _ BulkEditHeader - Default ⋅ Storybook

@ahmad-demos ahmad-demos added the documentation Improvements or additions to documentation label Oct 25, 2023
@ahmad-demos ahmad-demos marked this pull request as ready for review November 1, 2023 11:58
src/components/core/DpBulkEditHeader.stories.mdx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.mdx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.mdx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.mdx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.tsx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.tsx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.tsx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.tsx Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.vue Outdated Show resolved Hide resolved
src/components/shared/translations.js Outdated Show resolved Hide resolved
src/components/core/DpBulkEditHeader.stories.mdx Outdated Show resolved Hide resolved
Co-authored-by: spiess-demos <40452344+spiess-demos@users.noreply.github.com>
in fact, the checkbox above the item checkboxes should select
all items only if not all items are checked already. If all
items are checked, it should uncheck all items.

to stop tslint from complaining about undefined args,
(args as any) is used here.
@spiess-demos spiess-demos dismissed their stale review November 2, 2023 14:34

committed something here, myself

@spiess-demos spiess-demos requested review from salisdemos and removed request for spiess-demos November 2, 2023 14:35
@ahmad-demos ahmad-demos requested review from salisdemos and removed request for salisdemos November 2, 2023 14:39
Copy link
Contributor

@salisdemos salisdemos left a comment

Choose a reason for hiding this comment

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

Just one little thing

src/components/core/DpBulkEditHeader.vue Outdated Show resolved Hide resolved
@ahmad-demos ahmad-demos merged commit 09fd533 into main Nov 3, 2023
2 checks passed
@ahmad-demos ahmad-demos deleted the f_T34968_add_DpBulkEditHeader_to_storybook branch November 3, 2023 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants