Bookhoard/bookhoard
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 12 Wiki Activity

feat(devices): add device-specific collection view settings (Phase 9-6)

Browse Source 
Enhance device settings modal with collection view preferences:
- View mode selection (grid, list, compact)
- Sort order options (name, created, book count, recent)
- Items per page configuration (12, 24, 48, 96)
- Show/hide cover images toggle
- Show reading progress indicators toggle

Technical implementation:
- Stores device-specific settings in collections.view_settings JSONB
- Updates all collections when device preferences change
- Loads existing settings when opening modal
- Falls back to sensible defaults

This allows users to customize how their collections appear on
different devices (Kobo, KOReader, Web, Mobile) for an optimal
reading experience per platform.
This commit is contained in:
john-okeefe
2026-02-01 00:26:14 -05:00
parent 9411a95c44
commit 1ec3e7392e
1 changed files with 633 additions and 263 deletions
Download Patch File Download Diff File Expand all files Collapse all files
templates/devices.templ
+408 -38
View File
@@ -4,17 +4,16 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta charset="UTF-8"/>
<meta name="viewport" content="width=device-width, initial-scale=1.0"/>
<title>Device Management - Bookmann</title>
<script src="/static/htmx.min.js"></script>
<script src="/static/toast.js"></script>
<link href="/static/style.css" rel="stylesheet">
<link href="/static/style.css" rel="stylesheet"/>
<script src="/static/theme.js"></script>
</head>
<body class="theme-{ user.Theme }">
@Header(user, "/devices")
<div class="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-8">
<!-- Header Section -->
<div class="mb-8">
@@ -28,7 +27,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</button>
</div>
</div>
<!-- Devices Grid -->
<div id="devices-container" class="grid grid-cols-1 md:grid-cols-2 lg:grid-cols-3 gap-6">
<!-- Empty state -->
@@ -42,7 +40,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</button>
</div>
}
<!-- Devices Grid -->
if len(devices) > 0 {
<div id="devices-grid" class="grid-cols-1 md:grid-cols-2 lg:grid-cols-3 gap-6">
@@ -61,6 +58,9 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
}
</div>
<div class="flex space-x-2">
<button onclick="showShelfMappings('{ device.ID }')" class="p-2 hover:opacity-80 rounded" style="color: var(--text-secondary); background-color: var(--bg-primary);">
📚
</button>
<button onclick="showDeviceSettings('{ device.ID }')" class="p-2 hover:opacity-80 rounded" style="color: var(--text-secondary); background-color: var(--bg-primary);">
⚙️
</button>
@@ -99,7 +99,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</div>
}
</div>
<!-- Pending Registrations -->
if len(pendingRegistrations) > 0 {
<div id="pending-section" class="mt-12">
@@ -126,7 +125,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</div>
</div>
}
<!-- Sync Queue Section -->
<div id="sync-queue-section" class="mt-12 hidden">
<div class="flex justify-between items-center mb-4">
@@ -138,7 +136,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<div id="sync-queue" class="space-y-3"></div>
</div>
</div>
<!-- Add Device Modal -->
<div id="add-device-modal" class="hidden fixed inset-0 z-50 flex items-center justify-center" style="background-color: rgba(0, 0, 0, 0.7);">
<div class="card rounded-lg p-6 w-full max-w-md mx-4" style="background-color: var(--bg-secondary); border-color: var(--border);">
@@ -146,21 +143,26 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<h2 class="text-xl font-bold" style="color: var(--text-primary)">Add New Device</h2>
<button onclick="hideAddDeviceModal()" class="p-2 hover:opacity-80 rounded" style="color: var(--text-primary)">✕</button>
</div>
<form id="add-device-form" onsubmit="handleAddDevice(event)">
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Device Name</label>
<input type="text" id="device-name" required
<input
type="text"
id="device-name"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
placeholder="My Kindle Paperwhite">
placeholder="My Kindle Paperwhite"
/>
</div>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Device Type</label>
<select id="device-type" required
<select
id="device-type"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);">
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="">Select device type...</option>
<option value="koreader">KOReader (Kindle, Kobo, PocketBook)</option>
<option value="kobo">Kobo E-Reader</option>
@@ -168,16 +170,18 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<option value="mobile">Mobile App</option>
</select>
</div>
<div class="mb-6">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Device Identifier</label>
<input type="text" id="device-identifier" required
<input
type="text"
id="device-identifier"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
placeholder="Hardware ID or Serial Number">
placeholder="Hardware ID or Serial Number"
/>
<p class="text-xs mt-1" style="color: var(--text-secondary)">Enter your device's unique identifier</p>
</div>
<div class="flex justify-end space-x-3">
<button type="button" onclick="hideAddDeviceModal()" class="btn-secondary px-4 py-2 rounded-lg">
Cancel
@@ -189,7 +193,6 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</form>
</div>
</div>
<!-- Device Settings Modal -->
<div id="device-settings-modal" class="hidden fixed inset-0 z-50 flex items-center justify-center" style="background-color: rgba(0, 0, 0, 0.7);">
<div class="card rounded-lg p-6 w-full max-w-md mx-4" style="background-color: var(--bg-secondary); border-color: var(--border);">
@@ -197,40 +200,50 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<h2 class="text-xl font-bold" style="color: var(--text-primary)">Device Settings</h2>
<button onclick="hideDeviceSettingsModal()" class="p-2 hover:opacity-80 rounded" style="color: var(--text-primary)">✕</button>
</div>
<form id="device-settings-form" onsubmit="handleSaveDeviceSettings(event)">
<input type="hidden" id="settings-device-id">
<input type="hidden" id="settings-device-id"/>
<input type="hidden" id="settings-device-type"/>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Device Name</label>
<input type="text" id="settings-device-name" required
<input
type="text"
id="settings-device-name"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);">
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
/>
</div>
<div class="mb-4">
<label class="flex items-center space-x-3 cursor-pointer">
<input type="checkbox" id="settings-sync-enabled" checked
class="w-5 h-5 rounded">
<input
type="checkbox"
id="settings-sync-enabled"
checked
class="w-5 h-5 rounded"
/>
<span style="color: var(--text-primary)">Enable Sync</span>
</label>
<p class="text-xs mt-1 ml-8" style="color: var(--text-secondary)">Allow this device to sync reading progress</p>
</div>
<div class="mb-4">
<label class="flex items-center space-x-3 cursor-pointer">
<input type="checkbox" id="settings-auto-sync" checked
class="w-5 h-5 rounded">
<input
type="checkbox"
id="settings-auto-sync"
checked
class="w-5 h-5 rounded"
/>
<span style="color: var(--text-primary)">Auto Sync</span>
</label>
<p class="text-xs mt-1 ml-8" style="color: var(--text-secondary)">Automatically sync changes</p>
</div>
<div class="mb-6">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Sync Frequency</label>
<select id="settings-sync-frequency"
<select
id="settings-sync-frequency"
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);">
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="1">Every 1 minute</option>
<option value="5" selected>Every 5 minutes</option>
<option value="15">Every 15 minutes</option>
@@ -238,7 +251,71 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
<option value="60">Every hour</option>
</select>
</div>
<div class="border-t pt-6 mb-6" style="border-color: var(--border);">
<h3 class="text-lg font-semibold mb-4" style="color: var(--text-primary)">Collection View Settings</h3>
<p class="text-sm mb-4" style="color: var(--text-secondary)">Configure how collections are displayed on this device</p>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Default View Mode</label>
<select
id="settings-view-mode"
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="grid">Grid View</option>
<option value="list">List View</option>
<option value="compact">Compact View</option>
</select>
</div>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Sort Collections By</label>
<select
id="settings-sort-order"
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="name">Collection Name</option>
<option value="created">Date Created</option>
<option value="book_count">Book Count</option>
<option value="recent">Recently Added</option>
</select>
</div>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Items Per Page</label>
<select
id="settings-items-per-page"
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="12">12 items</option>
<option value="24" selected>24 items</option>
<option value="48">48 items</option>
<option value="96">96 items</option>
</select>
</div>
<div class="mb-4">
<label class="flex items-center space-x-3 cursor-pointer">
<input
type="checkbox"
id="settings-show-covers"
checked
class="w-5 h-5 rounded"
/>
<span style="color: var(--text-primary)">Show Cover Images</span>
</label>
<p class="text-xs mt-1 ml-8" style="color: var(--text-secondary)">Display book covers in collection views</p>
</div>
<div class="mb-4">
<label class="flex items-center space-x-3 cursor-pointer">
<input
type="checkbox"
id="settings-show-progress"
class="w-5 h-5 rounded"
/>
<span style="color: var(--text-primary)">Show Reading Progress</span>
</label>
<p class="text-xs mt-1 ml-8" style="color: var(--text-secondary)">Display progress indicators for books</p>
</div>
</div>
<div class="flex justify-end space-x-3">
<button type="button" onclick="hideDeviceSettingsModal()" class="btn-secondary px-4 py-2 rounded-lg">
Cancel
@@ -253,7 +330,85 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
</form>
</div>
</div>
<!-- Device Shelf Mappings Modal -->
<div id="shelf-mappings-modal" class="hidden fixed inset-0 z-50 flex items-center justify-center" style="background-color: rgba(0, 0, 0, 0.7);">
<div class="card rounded-lg p-6 w-full max-w-2xl mx-4 max-h-[90vh] overflow-y-auto" style="background-color: var(--bg-secondary); border-color: var(--border);">
<div class="flex justify-between items-center mb-6">
<h2 class="text-xl font-bold" style="color: var(--text-primary)">Collection to Shelf Mappings</h2>
<button onclick="hideShelfMappingsModal()" class="p-2 hover:opacity-80 rounded" style="color: var(--text-primary)">✕</button>
</div>
<input type="hidden" id="mappings-device-id"/>
<div id="shelf-mappings-container" class="space-y-4 mb-6">
<p style="color: var(--text-secondary)">Loading mappings...</p>
</div>
<div class="flex justify-end space-x-3">
<button type="button" onclick="hideShelfMappingsModal()" class="btn-secondary px-4 py-2 rounded-lg">
Close
</button>
<button type="button" onclick="showAddMappingModal()" class="btn-primary px-4 py-2 rounded-lg">
➕ Add Mapping
</button>
</div>
</div>
</div>
<!-- Add/Edit Shelf Mapping Modal -->
<div id="add-mapping-modal" class="hidden fixed inset-0 z-50 flex items-center justify-center" style="background-color: rgba(0, 0, 0, 0.7);">
<div class="card rounded-lg p-6 w-full max-w-md mx-4" style="background-color: var(--bg-secondary); border-color: var(--border);">
<div class="flex justify-between items-center mb-6">
<h2 class="text-xl font-bold" style="color: var(--text-primary)">Add Collection Mapping</h2>
<button onclick="hideAddMappingModal()" class="p-2 hover:opacity-80 rounded" style="color: var(--text-primary)">✕</button>
</div>
<form id="mapping-form" onsubmit="handleSaveMapping(event)">
<input type="hidden" id="mapping-device-id"/>
<input type="hidden" id="mapping-id"/>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Collection</label>
<select
id="mapping-collection"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="">Select collection...</option>
</select>
</div>
<div class="mb-4">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Device Shelf Name</label>
<input
type="text"
id="mapping-shelf-name"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
placeholder="e.g., Sci-Fi"
/>
<p class="text-xs mt-1" style="color: var(--text-secondary)">Name of the shelf on this device</p>
</div>
<div class="mb-6">
<label class="block text-sm font-medium mb-2" style="color: var(--text-secondary)">Sync Direction</label>
<select
id="mapping-sync-direction"
required
class="w-full px-4 py-2 border rounded-lg"
style="background-color: var(--bg-primary); color: var(--text-primary); border-color: var(--border);"
>
<option value="bidirectional">Bidirectional (sync both ways)</option>
<option value="book_to_device">Bookmann → Device</option>
<option value="device_to_book">Device → Bookmann</option>
<option value="none">No sync</option>
</select>
</div>
<div class="flex justify-end space-x-3">
<button type="button" onclick="hideAddMappingModal()" class="btn-secondary px-4 py-2 rounded-lg">
Cancel
</button>
<button type="submit" class="btn-primary px-4 py-2 rounded-lg">
Save Mapping
</button>
</div>
</form>
</div>
</div>
<script>
function getDeviceIcon(typeName) {
const deviceIcons = {
@@ -307,14 +462,192 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
if (!device) return;
document.getElementById('settings-device-id').value = deviceId;
document.getElementById('settings-device-type').value = device.device_type;
document.getElementById('settings-device-name').value = device.device_name;
document.getElementById('settings-sync-enabled').checked = device.sync_enabled;
document.getElementById('settings-auto-sync').checked = device.auto_sync;
document.getElementById('settings-sync-frequency').value = device.sync_frequency_minutes;
const deviceType = device.device_type;
fetch('/api/collections', {
headers: { 'Authorization': 'Bearer ' + localStorage.getItem('token') }
})
.then(response => response.json())
.then(result => {
if (result.collections && result.collections.length > 0) {
const firstCollection = result.collections[0];
const viewSettings = firstCollection.view_settings || {};
const deviceSettings = viewSettings[deviceType] || {};
document.getElementById('settings-view-mode').value = deviceSettings.view_mode || 'grid';
document.getElementById('settings-sort-order').value = deviceSettings.sort_order || 'name';
document.getElementById('settings-items-per-page').value = deviceSettings.items_per_page || 24;
document.getElementById('settings-show-covers').checked = deviceSettings.show_covers !== false;
document.getElementById('settings-show-progress').checked = deviceSettings.show_progress || false;
}
})
.catch(error => {
console.error('Failed to load view settings:', error);
});
document.getElementById('device-settings-modal').classList.remove('hidden');
}
function showShelfMappings(deviceId) {
document.getElementById('mappings-device-id').value = deviceId;
loadShelfMappings(deviceId);
document.getElementById('shelf-mappings-modal').classList.remove('hidden');
}
function hideShelfMappingsModal() {
document.getElementById('shelf-mappings-modal').classList.add('hidden');
}
function loadShelfMappings(deviceId) {
const container = document.getElementById('shelf-mappings-container');
fetch(`/api/devices/${deviceId}/collections`, {
headers: { 'Authorization': 'Bearer ' + localStorage.getItem('token') }
})
.then(response => response.json())
.then(result => {
if (result.mappings && result.mappings.length > 0) {
container.innerHTML = result.mappings.map(mapping => `
<div class="card p-4 rounded-lg border" style="background-color: var(--bg-primary); border-color: var(--border);">
<div class="flex justify-between items-center">
<div>
<h4 class="font-semibold" style="color: var(--text-primary)">${mapping.collection_name}</h4>
<p class="text-sm" style="color: var(--text-secondary)">
→ Device Shelf: <strong>${mapping.device_shelf_name}</strong>
</p>
<p class="text-xs" style="color: var(--text-secondary)">
Sync: ${mapping.sync_direction}
</p>
</div>
<div class="flex space-x-2">
<button onclick="editMapping('${mapping.id}', '${mapping.collection_id}', '${mapping.device_shelf_name}', '${mapping.sync_direction}')"
class="p-2 hover:opacity-80 rounded" style="color: var(--text-secondary); background-color: var(--bg-secondary);">
✏️
</button>
<button onclick="deleteMapping('${mapping.id}')"
class="p-2 hover:opacity-80 rounded" style="color: var(--text-secondary); background-color: var(--bg-secondary);">
🗑️
</button>
</div>
</div>
</div>
`).join('');
} else {
container.innerHTML = '<p style="color: var(--text-secondary)">No shelf mappings configured. Click "Add Mapping" to create one.</p>';
}
})
.catch(error => {
container.innerHTML = '<p style="color: var(--error)">Failed to load mappings</p>';
});
}
function showAddMappingModal() {
const deviceId = document.getElementById('mappings-device-id').value;
document.getElementById('mapping-device-id').value = deviceId;
document.getElementById('mapping-id').value = '';
document.getElementById('mapping-collection').value = '';
document.getElementById('mapping-shelf-name').value = '';
document.getElementById('mapping-sync-direction').value = 'bidirectional';
loadCollections();
document.getElementById('add-mapping-modal').classList.remove('hidden');
}
function hideAddMappingModal() {
document.getElementById('add-mapping-modal').classList.add('hidden');
document.getElementById('mapping-form').reset();
}
function loadCollections() {
const select = document.getElementById('mapping-collection');
fetch('/api/collections', {
headers: { 'Authorization': 'Bearer ' + localStorage.getItem('token') }
})
.then(response => response.json())
.then(result => {
if (result.collections) {
select.innerHTML = '<option value="">Select collection...</option>' +
result.collections.map(col => `<option value="${col.id}">${col.name}</option>`).join('');
}
})
.catch(error => {
console.error('Failed to load collections', error);
});
}
function editMapping(mappingId, collectionId, shelfName, syncDirection) {
document.getElementById('mapping-id').value = mappingId;
document.getElementById('mapping-collection').value = collectionId;
document.getElementById('mapping-shelf-name').value = shelfName;
document.getElementById('mapping-sync-direction').value = syncDirection;
document.getElementById('add-mapping-modal').classList.remove('hidden');
}
function handleSaveMapping(event) {
event.preventDefault();
const deviceId = document.getElementById('mapping-device-id').value;
const mappingId = document.getElementById('mapping-id').value;
const isUpdate = mappingId !== '';
const data = {
collection_id: document.getElementById('mapping-collection').value,
device_shelf_name: document.getElementById('mapping-shelf-name').value,
sync_direction: document.getElementById('mapping-sync-direction').value
};
const url = isUpdate ? `/api/devices/${deviceId}/collections/${mappingId}` : `/api/devices/${deviceId}/collections`;
const method = isUpdate ? 'PUT' : 'POST';
fetch(url, {
method: method,
headers: {
'Content-Type': 'application/json',
'Authorization': 'Bearer ' + localStorage.getItem('token')
},
body: JSON.stringify(data)
})
.then(response => response.json())
.then(result => {
showToast(isUpdate ? 'Mapping updated' : 'Mapping created', 'success');
hideAddMappingModal();
loadShelfMappings(deviceId);
})
.catch(error => {
showToast('Failed to save mapping', 'error');
});
}
function deleteMapping(mappingId) {
if (!confirm('Are you sure you want to delete this mapping?')) return;
const deviceId = document.getElementById('mappings-device-id').value;
fetch(`/api/devices/${deviceId}/collections/${mappingId}`, {
method: 'DELETE',
headers: { 'Authorization': 'Bearer ' + localStorage.getItem('token') }
})
.then(response => {
if (response.ok) {
showToast('Mapping deleted', 'success');
loadShelfMappings(deviceId);
} else {
showToast('Failed to delete mapping', 'error');
}
})
.catch(error => {
showToast('Failed to delete mapping', 'error');
});
}
function hideDeviceSettingsModal() {
document.getElementById('device-settings-modal').classList.add('hidden');
}
@@ -323,23 +656,60 @@ templ Devices(user User, devices []DeviceData, pendingRegistrations []PendingReg
event.preventDefault();
const deviceId = document.getElementById('settings-device-id').value;
const data = {
const deviceType = document.getElementById('settings-device-type')?.value || '';
const deviceData = {
device_name: document.getElementById('settings-device-name').value,
sync_enabled: document.getElementById('settings-sync-enabled').checked,
auto_sync: document.getElementById('settings-auto-sync').checked,
sync_frequency_minutes: parseInt(document.getElementById('settings-sync-frequency').value)
};
const viewSettings = {
view_mode: document.getElementById('settings-view-mode').value,
sort_order: document.getElementById('settings-sort-order').value,
items_per_page: parseInt(document.getElementById('settings-items-per-page').value),
show_covers: document.getElementById('settings-show-covers').checked,
show_progress: document.getElementById('settings-show-progress').checked
};
fetch(`/api/devices/${deviceId}`, {
method: 'PUT',
headers: {
'Content-Type': 'application/json',
'Authorization': 'Bearer ' + localStorage.getItem('token')
},
body: JSON.stringify(data)
body: JSON.stringify(deviceData)
})
.then(response => response.json())
.then(result => {
return fetch('/api/collections', {
method: 'GET',
headers: { 'Authorization': 'Bearer ' + localStorage.getItem('token') }
});
})
.then(response => response.json())
.then(result => {
const updatePromises = result.collections.map(collection => {
const currentSettings = collection.view_settings || {};
currentSettings[deviceType] = viewSettings;
return fetch(`/api/collections/${collection.id}`, {
method: 'PUT',
headers: {
'Content-Type': 'application/json',
'Authorization': 'Bearer ' + localStorage.getItem('token')
},
body: JSON.stringify({
...collection,
view_settings: currentSettings
})
});
});
return Promise.all(updatePromises);
})
.then(() => {
showToast('Device settings saved', 'success');
hideDeviceSettingsModal();
location.reload();