Skip to content

Commit

Permalink
fix django-cms#363: Better UX in versioning listview (django-cms#364)
Browse files Browse the repository at this point in the history
Co-authored-by: Fabian Braun <[email protected]>
  • Loading branch information
jrief and fsbraun authored Mar 5, 2024
1 parent 8a4e6c9 commit 93a3e52
Show file tree
Hide file tree
Showing 2 changed files with 40 additions and 0 deletions.
3 changes: 3 additions & 0 deletions djangocms_versioning/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -608,6 +608,9 @@ class VersionAdmin(ChangeListActionsMixin, admin.ModelAdmin, metaclass=MediaDefi
# def get_queryset(self, request):
# return super().get_queryset(request).prefetch_related('content')

class Media:
js = ["djangocms_versioning/js/versioning.js"]

def get_changelist(self, request, **kwargs):
return VersionChangeList

Expand Down
37 changes: 37 additions & 0 deletions djangocms_versioning/static/djangocms_versioning/js/versioning.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
(function() {
var firstChecked, lastChecked;

function handleVersionSelection(event) {
if (firstChecked instanceof HTMLInputElement && firstChecked.checked) {
firstChecked.checked = false;
firstChecked.closest('tr').classList.remove('selected');
firstChecked = lastChecked;
}
if (event.target instanceof HTMLInputElement) {
if (event.target.checked) {
firstChecked = lastChecked;
lastChecked = event.target;
} else if (firstChecked === event.target) {
firstChecked = null;
} else {
lastChecked = null;
}
}
}

document.addEventListener('DOMContentLoaded', function(){
var selectedVersions = document.querySelectorAll('#result_list input[type="checkbox"].action-select');
var selectElement = document.querySelector('#changelist-form select[name="action"]');
if (selectElement instanceof HTMLSelectElement) {
for (var i = 0; i < selectElement.options.length; i++) {
if (selectElement.options[i].value && selectElement.options[i].value !== 'compare_versions') {
// for future safety: do not restrict on two selected versions, since there might be other actions
return;
}
}
}
selectedVersions.forEach(function(selectedVersion){
selectedVersion.addEventListener('change', handleVersionSelection);
});
});
})();

0 comments on commit 93a3e52

Please sign in to comment.