⚡ Bolt: Pre-calculate search strings and formats for optimized rendering#56
⚡ Bolt: Pre-calculate search strings and formats for optimized rendering#56MrAlokTech wants to merge 1 commit intomainfrom
Conversation
Pre-calculate search properties and format dates inside loadPDFDatabase to avoid redundant calculations during loop iterations in renderPDFs and createPDFCard. Co-authored-by: MrAlokTech <107493955+MrAlokTech@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Deploying classnotes with
|
| Latest commit: |
d5d2e32
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://b872a9de.classnotes.pages.dev |
| Branch Preview URL: | https://bolt-performance-pre-indexin.classnotes.pages.dev |
💡 What: Added a
prepareSearchIndexfunction that executes once after loading the PDF dataset. This function computes_searchStr,_isNew, and_formattedDatefor each item. ModifiedrenderPDFsandcreatePDFCardto rely on these pre-computed properties instead of doing repetitive Date processing and string concatenation in the render loop.🎯 Why: Previously,
renderPDFswould calculatetoLowerCase()on multiple properties and parse newDateobjects inside loops every time a filter changed, keystroke happened, or a new semester was selected. By pre-indexing these values when data loads, the filter and render loops become dramatically faster, significantly reducing CPU usage during heavy user interactions.📊 Impact: Expected to reduce filter iteration time by removing multiple redundant
.toLowerCase()and.includes()operations, and rendering speed is improved by bypassingIntl.DateTimeFormatevaluations inside inner loops, optimizing UI responsiveness on lower-end devices.🔬 Measurement: Check the UI interactions (search filtering, scrolling, changing tabs) and verify they are smoother. The logic has been explicitly reviewed and maintains exactly the previous behavior.
PR created automatically by Jules for task 12142998602662649987 started by @MrAlokTech