Feature/lucashlaing/opensource website#174
Open
lucashlaing wants to merge 58 commits intoWill-Hsu:mainfrom
Open
Feature/lucashlaing/opensource website#174lucashlaing wants to merge 58 commits intoWill-Hsu:mainfrom
lucashlaing wants to merge 58 commits intoWill-Hsu:mainfrom
Conversation
…orOption Removed Minor Option When Minor is Unadded Will-Hsu#167
…Animation Home and About Page Animation
Event Card UI & Rewards Hover
Bug/backend/mailchimp
…rdPhotos adding board photos and remove innovate org
…ntFix Login Page Alignment Issues
…1/FE-mobileFormatting
…1/FE-mobileFormatting
…eFormatting Feature/ganeshkumar321/fe mobile formatting
…1/FE-mobileFormatting Revert "Feature/ganeshkumar321/fe mobile formatting"
fixed small syntax errors (semicolons and single-quotes)
… to the center, and matched other buttons to the same style as other page
…eFormatting fixed some frontend mobile issues
Feature/SithuSoe04/fe-team-update
…xtUpdate Temporarily added Shruti and Himansi as admins
Fixed minor margin issues
…/FE-TemplateWebsite Added Template Website for the Communities
Since the issue **always skips the letter at index 1**, and the console logs show the correct indexing logic, let's focus on **how state updates are processed inside React's reconciliation system**. ### **Key Observations:** 1. **The first letter is always correct.** 2. **The second letter is always skipped.** 3. **The rest of the letters appear correctly.** 4. **No matter what text you pass in, the second character always disappears.** --- ## **Likely Root Cause:** ### **React's batching updates incorrectly due to async state updates.** - The issue occurs because **relies on **. - React batches multiple state updates inside , which may **cause the second letter to be skipped**. ### **Why?** - The function schedules updates asynchronously. - If React batches multiple updates, may not update in time before the next iteration. - **Since is inside , the state might not reflect the latest changes in time for the second character.** --- ## **Final Fix: Use Functional Updates with a Ref** We'll **use a ref** to track synchronously **instead of relying on state updates**. ### **Updated Code:** --- ## **Why This Fix Works** ✅ **Prevents skipping issues** by using instead of relying on . ✅ **Ensures consistent updates** because refs update immediately, avoiding batching delays. ✅ **Fixes async state issues** where the second letter gets skipped due to React's state batching. ### **Next Steps** 1. **Replace your existing with this version.** 2. **Test with and .** 3. **Verify if the second letter is still skipped.** This should resolve your issue! � Let me know how it works.
…nitiesPage-MemberButtonDebug opportunities page bug
…2025-feb Fixed About page bug at "WHAT IS CSES?", minor spelling checks across pages
…feature/lucashlaing/OpensourceWebsite
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
finished open source page