Skip to content

Commit

Permalink
Merge pull request #19242 from jmchilton/improve_empty_message
Browse files Browse the repository at this point in the history
[24.2] Improve the empty history message for guiding users to collections.
  • Loading branch information
ahmedhamidawan authored Dec 4, 2024
2 parents ef82f8f + 82b410a commit f5cb35c
Show file tree
Hide file tree
Showing 5 changed files with 659 additions and 696 deletions.
9 changes: 3 additions & 6 deletions client/src/components/Collections/CollectionCreatorModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -260,9 +260,6 @@ function resetModal() {
<BAlert v-else-if="!fromSelection && historyItemsError" variant="danger" show>
{{ historyItemsError }}
</BAlert>
<BAlert v-else-if="!creatorItems?.length" variant="info" show>
{{ localize("No items available to create a collection.") }}
</BAlert>
<BAlert v-else-if="creatingCollection" variant="info" show>
<LoadingSpan :message="localize('Creating collection')" />
</BAlert>
Expand All @@ -289,7 +286,7 @@ function resetModal() {
<ListCollectionCreator
v-else-if="props.collectionType === 'list'"
:history-id="props.historyId"
:initial-elements="creatorItems"
:initial-elements="creatorItems || []"
:default-hide-source-items="props.defaultHideSourceItems"
:from-selection="fromSelection"
:extensions="props.extensions"
Expand All @@ -298,7 +295,7 @@ function resetModal() {
<PairedListCollectionCreator
v-else-if="props.collectionType === 'list:paired'"
:history-id="props.historyId"
:initial-elements="creatorItems"
:initial-elements="creatorItems || []"
:default-hide-source-items="props.defaultHideSourceItems"
:from-selection="fromSelection"
:extensions="props.extensions"
Expand All @@ -307,7 +304,7 @@ function resetModal() {
<PairCollectionCreator
v-else-if="props.collectionType === 'paired'"
:history-id="props.historyId"
:initial-elements="creatorItems"
:initial-elements="creatorItems || []"
:default-hide-source-items="props.defaultHideSourceItems"
:from-selection="fromSelection"
:extensions="props.extensions"
Expand Down
Loading

0 comments on commit f5cb35c

Please sign in to comment.