Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/optimizations #17

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

godcl1623
Copy link

Made some changes to optimize the app.

  1. Make interactive graph properly renewed on page navigation
    Used next/dynamic to dynamically import CytoscapeComponent component.

  2. Fixed object id changes on every navigation
    FolderTree has folded on every second page move, possible reason seems to be object id change, since _counter variable's value incremented on every page move.
    Changed object id by thisObject.name, it seems FolderTree works well.

  3. Add cache to increase build speed
    If number of posts increased over hundreds, build speed slowed down(e.g. about 9 minutes for 300+ files).
    By replacing convertObject(getDirectoryData()) in getStaticProps to cache, build speed increased(e.g. about 27 seconds for 300+ files).

@netlify
Copy link

netlify bot commented Feb 15, 2023

Deploy Preview for digital-garden-demo ready!

Name Link
🔨 Latest commit 2025833
🔍 Latest deploy log https://app.netlify.com/sites/digital-garden-demo/deploys/63ed0f39c9238b0007914ae8
😎 Deploy Preview https://deploy-preview-17--digital-garden-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

mrdrprofuroboros pushed a commit to mrdrprofuroboros/digital-garden that referenced this pull request Dec 23, 2023
* feat: add kotlin

* chore: import kotlin project as library

* chore: add kotlin-markdown parser

* chore: add lock file

* chore: add ktlint

* ci: add check-kotlin job

* ci: rename build job

* feat: add TagConsumer

* feat: add NodeProcessor

* feat: add CustomProcessors

* feat: add ReactElementGenerator

* wip: add Processor

* style: format code

* style: format and fix documents

* refactor: remove test codes

* style: format code

* feat: add generics modifiers

* refactor: move impl processors

* feat: implement processors

* wip: add Processors

* refactor: rename function

* feat: create markdown parsers

* ci: add gradle and java setup task

* feat: export markdown parsers

* fix: update test files

* style: format codes

* ci: add action write permission

* ci: remove redundant permission settings

* style: rename target visitor class

* feat: implement AutoLink todo processes

* feat: implement Processors

* feat: implement visitLeaf fuction

* fix: incorrect tag process

* fix: add missed args

* chore: add kotlin-react-test library

* fix: code block contents

* refactor: rename Processor

* fix: incorrect trimming process

* fix: incorrect element merging process

* fix: incorrect line breaking rules

* fix: incorrect separator string

* fix: incorrect tag process

* fix: HTML content process

* fix: link title process

* test: add useful functions

* wip: add ReactElementGenerator test(50/652)

* refactor: change consume process

* fix: incorrect tag process order

* wip: add ReactElementGenerator test(100/652)

* wip: add ReactElementGenerator test(120/652)

* fix: mistaken type pair

* fix: remove redundant linebreak

* wip: add ReactElementGenerator test(200/652)

* wip: add ReactElementGenerator test(250/652)

* wip: add ReactElementGenerator test(300/652)

* wip: add ReactElementGenerator test(350/652)

* wip: add ReactElementGenerator test(400/652)

* fix: add null check in link title injection

* wip: add ReactElementGenerator test(500/652)

* fix: add null check in image title injection

* wip: add ReactElementGenerator test(600/652)

* test: add ReactElementGenerator test(652/652)

* fix: link map initialization process

* build: add stacktrace option in kotlin build task

* chore: update react

* chore: fix docker build script

* ci: add docker build task

* feat: implement obsidian link parser

* style: rename fakeNode -> slimNode

* style: format code

* fix: remove whitespace from linkText

* docs: fix comment

* feat: add ObsidianLineBreakParser

* chore: update kotlinter

* style: format code

* refactor: fix link text process

* test: add obsidian link test

* feat: add RoutableProps

* feat: add Cache

* fix: replace markdown content analyzing system and optimise treeData conversion

* fix: replace parent element body to div

body not allowed in MdComponent

* fix: remove content cache

function cannot store

* fix: includes image/canvas files

* refactor: optimise markdown/fileTree analysing system

* fix: deals with fileName duplication case

* chore: add KStore/serialization/coroutine library

* fix: rework data flow

* fix: router property data flow

* refactor: add type information

* chore: add ignore targets

* refactor: rework backLink system

* fix: backlink click behavior

* refactor: rework graph system

* refactor: replace toFilePath function

* refactor: remove unused codes

* refactor: improve type safety

* chore: remove file that is no longer generated

* fix: support for some test case

* feat: implement code fence syntax highlighting

* fix: cover direct link conversion

* fix: dependency list duplication

* fix: backlink NullPointer bug

* fix: link resolver misidentifying URLs as internal links

* docs: add description of string conversion

* fix: for cases where there is no space between link and text

* style: fix typo

* fix: code block detection rule

* fix: add confirmation that the target is an .md file

* fix: replace markdown extension regex

* docs: add comments

* refactor: cache FolderTreeData in initCache

To preserve the original file structure while implementing the automatic movement of media files

* feat: add image file support

Also, no longer need to manually add media to the public file.

* style: rename cache file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant