Delete the phantom saved-views sidebar
.saved-views-sidebar was never in the DOM at any width. Saved views ship
as a modal (js/saved-views.js). Removes the two CSS rules that styled it,
the section 24 comment about a narrowing that was already deleted, the
always-null toggleSidebar() and its export, the menu:toggle_sidebar
listener, and the View > Toggle Sidebar menu item that drove it.
- Co-Authored-By
- Claude Opus 5 (1M context) <noreply@anthropic.com>
3 files changed,
+0 insertions,
-40 deletions
| 244 |
244 |
|
&work_submenu,
|
| 245 |
245 |
|
&time_submenu,
|
| 246 |
246 |
|
&messages_submenu,
|
| 247 |
|
- |
&PredefinedMenuItem::separator(app)?,
|
| 248 |
|
- |
&MenuItem::with_id(
|
| 249 |
|
- |
app,
|
| 250 |
|
- |
"toggle_sidebar",
|
| 251 |
|
- |
"Toggle Sidebar",
|
| 252 |
|
- |
true,
|
| 253 |
|
- |
Some("CmdOrCtrl+\\"),
|
| 254 |
|
- |
)?,
|
| 255 |
247 |
|
],
|
| 256 |
248 |
|
)?;
|
| 257 |
249 |
|
|
| 3386 |
3386 |
|
picked from the user agent, and now a window dragged under 600px gets the
|
| 3387 |
3387 |
|
compact treatment in §25 instead of this. */
|
| 3388 |
3388 |
|
@media (min-width: 600px) and (max-width: 1024px) {
|
| 3389 |
|
- |
/* No .saved-views-sidebar narrowing here. It set 180px and never took
|
| 3390 |
|
- |
effect: §46 sets 200px on the same specificity further down the file
|
| 3391 |
|
- |
and won. Deleted rather than repaired, so the sidebar is one width in
|
| 3392 |
|
- |
the wide shell and the stylesheet says so once. */
|
| 3393 |
|
- |
|
| 3394 |
3389 |
|
.filter-bar {
|
| 3395 |
3390 |
|
flex-wrap: wrap;
|
| 3396 |
3391 |
|
}
|
| 3439 |
3434 |
|
font-size: var(--font-size-3xl);
|
| 3440 |
3435 |
|
}
|
| 3441 |
3436 |
|
|
| 3442 |
|
- |
.saved-views-sidebar {
|
| 3443 |
|
- |
display: none;
|
| 3444 |
|
- |
}
|
| 3445 |
|
- |
|
| 3446 |
3437 |
|
.day-plan-content {
|
| 3447 |
3438 |
|
flex-direction: column;
|
| 3448 |
3439 |
|
}
|
| 5335 |
5326 |
|
}
|
| 5336 |
5327 |
|
|
| 5337 |
5328 |
|
/* 46. Saved Views Sidebar (desktop UI only, hidden on mobile) */
|
| 5338 |
|
- |
@media (min-width: 600px) {
|
| 5339 |
|
- |
.saved-views-sidebar {
|
| 5340 |
|
- |
width: 200px;
|
| 5341 |
|
- |
flex-shrink: 0;
|
| 5342 |
|
- |
background: var(--surface-raised);
|
| 5343 |
|
- |
border-right: var(--border-width) solid var(--border);
|
| 5344 |
|
- |
display: flex;
|
| 5345 |
|
- |
flex-direction: column;
|
| 5346 |
|
- |
overflow: hidden;
|
| 5347 |
|
- |
}
|
| 5348 |
|
- |
}
|
| 5349 |
|
- |
|
| 5350 |
5329 |
|
.button--icon {
|
| 5351 |
5330 |
|
background: none;
|
| 5352 |
5331 |
|
border: none;
|
| 163 |
163 |
|
listen('menu:view_day_plan', () => GoingsOn.navigation.switchView('day-plan'));
|
| 164 |
164 |
|
listen('menu:view_weekly_review', () => GoingsOn.navigation.switchView('weekly-review'));
|
| 165 |
165 |
|
listen('menu:view_monthly_review', () => GoingsOn.navigation.switchView('monthly-review'));
|
| 166 |
|
- |
listen('menu:toggle_sidebar', () => GoingsOn.app.toggleSidebar());
|
| 167 |
166 |
|
|
| 168 |
167 |
|
// Tools menu
|
| 169 |
168 |
|
listen('menu:sync_email', () => GoingsOn.app.syncAllEmailAccounts());
|
| 284 |
283 |
|
}
|
| 285 |
284 |
|
}
|
| 286 |
285 |
|
|
| 287 |
|
- |
// Sidebar Toggle
|
| 288 |
|
- |
|
| 289 |
|
- |
function toggleSidebar() {
|
| 290 |
|
- |
const sidebar = document.querySelector('.saved-views-sidebar');
|
| 291 |
|
- |
if (sidebar) {
|
| 292 |
|
- |
sidebar.classList.toggle('hidden');
|
| 293 |
|
- |
}
|
| 294 |
|
- |
}
|
| 295 |
|
- |
|
| 296 |
286 |
|
// Email Sync
|
| 297 |
287 |
|
|
| 298 |
288 |
|
async function syncAllEmailAccounts() {
|
| 400 |
390 |
|
}
|
| 401 |
391 |
|
|
| 402 |
392 |
|
GoingsOn.app = {
|
| 403 |
|
- |
toggleSidebar,
|
| 404 |
393 |
|
syncAllEmailAccounts,
|
| 405 |
394 |
|
openAboutModal,
|
| 406 |
395 |
|
refreshCurrentViewData,
|