Skip to content

Commit

Permalink
Merge pull request #220 from bcgov/bugfix/bulk-results-css
Browse files Browse the repository at this point in the history
Bulk results modal: tweak DataTable styling
  • Loading branch information
TimCsaky authored Jun 27, 2024
2 parents d56fa81 + 03e6e52 commit 0336a74
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
2 changes: 0 additions & 2 deletions frontend/src/components/common/BulkPermission.vue
Original file line number Diff line number Diff line change
Expand Up @@ -408,6 +408,4 @@ const onSubmit = handleSubmit(async (values: any, { resetForm }) => {
:resource="resource"
:resource-type="resourceType"
/>

<pre v-if="complete">{{ results }}</pre>
</template>
16 changes: 13 additions & 3 deletions frontend/src/components/common/BulkPermissionResults.vue
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@ const props = withDefaults(defineProps<Props>(), {
const modelValue = defineModel<boolean>({ default: false });
const exportFileName =
props.resourceType === 'object'
? props.resource.name.replace(/\.[^/.]+$/, '')
: `${props.resource.bucketName}_bulk_results`;
// Exports DataTable results
const batchResults = ref();
Expand All @@ -31,9 +36,9 @@ const exportCSV = () => {
>
<DataTable
ref="batchResults"
:export-filename="`${resourceType === 'object' ? resource.name.replace(/\.[^/.]+$/, '') : resource.bucketName}_bulk_results`"
:export-filename="exportFileName"
:value="props.results"
class="p-datatable-striped"
class="p-datatable-striped results-modal-table"
>
<div class="action-buttons">
<Button
Expand All @@ -54,7 +59,7 @@ const exportCSV = () => {
header="Result"
>
<template #body="{ data }">
<span>
<span class="mr-3">
<span class="m-1">
<font-awesome-icon
v-if="data.status === 1"
Expand Down Expand Up @@ -90,4 +95,9 @@ const exportCSV = () => {
.icon-noaction {
color: $bcbox-noaction;
}
.results-modal-table {
text-wrap: nowrap;
min-width: 35em;
}
</style>

0 comments on commit 0336a74

Please sign in to comment.