[PR #1458] [MERGED] initial changes for responsiveness #1363

Closed
opened 2026-03-03 00:25:50 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BoostIO/BoostNote-App/pull/1458
Author: @Davy-c
Created: 1/28/2022
Status: Merged
Merged: 1/31/2022
Merged by: @Davy-c

Base: masterHead: feat/ux-responsiveness


📝 Commits (10+)

📊 Changes

34 files changed (+688 additions, -371 deletions)

View changed files

📝 package-lock.json (+14 -0)
📝 package.json (+2 -0)
📝 src/cloud/components/App.tsx (+3 -5)
📝 src/cloud/components/Application.tsx (+13 -1)
📝 src/cloud/components/ApplicationContent.tsx (+0 -12)
src/cloud/components/ApplicationPageLoader.tsx (+41 -0)
📝 src/cloud/components/ApplicationTopbar.tsx (+1 -0)
src/cloud/components/ApplicationWithoutInfoLoader.tsx (+47 -0)
📝 src/cloud/components/DashboardPage/index.tsx (+6 -0)
📝 src/cloud/components/DocPage/index.tsx (+0 -12)
📝 src/cloud/components/DocPreview/DocPreviewRealtime.tsx (+2 -2)
📝 src/cloud/components/DocPreview/index.tsx (+2 -2)
📝 src/cloud/components/Editor/index.tsx (+2 -53)
📝 src/cloud/components/FolderPage/index.tsx (+3 -16)
📝 src/cloud/components/MarkdownView/index.tsx (+2 -1)
📝 src/cloud/components/Router.tsx (+66 -91)
📝 src/cloud/components/WorkspacePage/index.tsx (+16 -9)
📝 src/cloud/index.tsx (+4 -1)
📝 src/cloud/lib/stores/nav/store.tsx (+14 -0)
📝 src/cloud/lib/stores/pageStore/store.tsx (+12 -4)

...and 14 more files

📄 Description

  • fix HMR on dev: moved RouterProvider one step up to keep the state ( prevents reloading on previous page etc..)
  • Remove blocking global data fetch, added loaders to compensate
  • Remove blocking page data fetch, added loaders for ux
  • Remove fetch initial props on pageProps query change but with same pathname ( shallow push )
  • Remove blocking nav initial load check, added loaders in dashboard to compensate
  • Remove obsolete cache in resource page
  • Remove blocking editor data loading
  • Added loader for sidebar tree, breadcrumb, editor load, folder pages..

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BoostIO/BoostNote-App/pull/1458 **Author:** [@Davy-c](https://github.com/Davy-c) **Created:** 1/28/2022 **Status:** ✅ Merged **Merged:** 1/31/2022 **Merged by:** [@Davy-c](https://github.com/Davy-c) **Base:** `master` ← **Head:** `feat/ux-responsiveness` --- ### 📝 Commits (10+) - [`d8d9384`](https://github.com/BoostIO/BoostNote-App/commit/d8d9384c19cd652c11cedc8a5b13efa88d02c025) initial changes for responsiveness - [`557486d`](https://github.com/BoostIO/BoostNote-App/commit/557486d35429770bd2c383d965b3ac8a616dae0a) dashboards and some cleanup - [`0cc8cd5`](https://github.com/BoostIO/BoostNote-App/commit/0cc8cd5a64068c0189e75c04862533d8bee43668) remove pageprops just for change of query params - [`aa0a525`](https://github.com/BoostIO/BoostNote-App/commit/aa0a525761fd45ded97ce4489fa76b6d6531afc9) add short ids in all loaders - [`9399c47`](https://github.com/BoostIO/BoostNote-App/commit/9399c4724f7ec1269c0ad62f4970bf586ee5b39d) remove unused property - [`5639e9b`](https://github.com/BoostIO/BoostNote-App/commit/5639e9b596a54f9e03c41e8e85e288dbc7fdecb9) remove log - [`fb5fd57`](https://github.com/BoostIO/BoostNote-App/commit/fb5fd57f137122a27a90d40072e309b048568625) remove unused import - [`c359a7a`](https://github.com/BoostIO/BoostNote-App/commit/c359a7a33cf1c91b5f2d1b1da1a6e2ef6e043846) label outside of initial load - [`8b72949`](https://github.com/BoostIO/BoostNote-App/commit/8b72949acc70c85cd50c849c2a23d8b0e4b88c85) restore efficient hook dependency - [`b10b645`](https://github.com/BoostIO/BoostNote-App/commit/b10b6452fcea39b6b286e50b1caf4c0c54228d64) fix hmr ### 📊 Changes **34 files changed** (+688 additions, -371 deletions) <details> <summary>View changed files</summary> 📝 `package-lock.json` (+14 -0) 📝 `package.json` (+2 -0) 📝 `src/cloud/components/App.tsx` (+3 -5) 📝 `src/cloud/components/Application.tsx` (+13 -1) 📝 `src/cloud/components/ApplicationContent.tsx` (+0 -12) ➕ `src/cloud/components/ApplicationPageLoader.tsx` (+41 -0) 📝 `src/cloud/components/ApplicationTopbar.tsx` (+1 -0) ➕ `src/cloud/components/ApplicationWithoutInfoLoader.tsx` (+47 -0) 📝 `src/cloud/components/DashboardPage/index.tsx` (+6 -0) 📝 `src/cloud/components/DocPage/index.tsx` (+0 -12) 📝 `src/cloud/components/DocPreview/DocPreviewRealtime.tsx` (+2 -2) 📝 `src/cloud/components/DocPreview/index.tsx` (+2 -2) 📝 `src/cloud/components/Editor/index.tsx` (+2 -53) 📝 `src/cloud/components/FolderPage/index.tsx` (+3 -16) 📝 `src/cloud/components/MarkdownView/index.tsx` (+2 -1) 📝 `src/cloud/components/Router.tsx` (+66 -91) 📝 `src/cloud/components/WorkspacePage/index.tsx` (+16 -9) 📝 `src/cloud/index.tsx` (+4 -1) 📝 `src/cloud/lib/stores/nav/store.tsx` (+14 -0) 📝 `src/cloud/lib/stores/pageStore/store.tsx` (+12 -4) _...and 14 more files_ </details> ### 📄 Description - fix HMR on dev: moved RouterProvider one step up to keep the state ( prevents reloading on previous page etc..) - Remove blocking global data fetch, added loaders to compensate - Remove blocking page data fetch, added loaders for ux - Remove fetch initial props on pageProps query change but with same pathname ( shallow push ) - Remove blocking nav initial load check, added loaders in dashboard to compensate - Remove obsolete cache in resource page - Remove blocking editor data loading - Added loader for sidebar tree, breadcrumb, editor load, folder pages.. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 00:25:50 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/BoostNote-App#1363
No description provided.