Conversation
There was a problem hiding this comment.
- The scrolling issue still remains, we need #6 to fix it. Change title and update description.
- Captured an important bug: Masked input doesn't mask key code:
Looks great, but needs a lot of change, and I will add an issue for [masked] issue.
|
|
||
| <span class="cmt"><!-- 2. Load SDK --></span> | ||
| <span class="kw"><script</span> <span class="fn">src</span>=<span class="str">"tele.umd.min.js"</span><span class="kw">></script></span> | ||
| <!-- Live Logger & Heatmap Grid --> |
There was a problem hiding this comment.
This has an overflow problem on mobile like header.
| </div> | ||
|
|
||
| <div class="compare-section"> | ||
| <div class="table-container"> |
There was a problem hiding this comment.
This section margin for mobile is too high, so just ~2 columns of the table are visible at the same time.
| tele.<span class="fn">config</span>({ | ||
| <span class="fn">onFlush</span>: events => myApi.<span class="fn">post</span>(events) | ||
| }); | ||
| <!-- Documentation & Tabs Section --> |
There was a problem hiding this comment.
This section margin in too high like compare table, so other tabs are invisible and codes width is very low.
mkh-user
left a comment
There was a problem hiding this comment.
LGTM, but needs further optimization for mobile. Overflow and collapsed table issues still remain from the previous review.
mkh-user
left a comment
There was a problem hiding this comment.
Changes to fix demo issues in GitHub pages
| flex-direction: column; | ||
| transition: var(--transition-smooth); | ||
| } | ||
|
|
There was a problem hiding this comment.
| .panel-header { | |
| padding-bottom: 20px; | |
| } |
|
|
||
| <!-- Load SDK --> | ||
| <script src="../dist/tele.umd.js" | ||
| onerror="this.onerror=null; this.src='https://cdn.jsdelivr.net/npm/telesense/dist/tele.umd.min.js';"></script> |
There was a problem hiding this comment.
Fallback to real demo distribution file instead of Telesense release:
| onerror="this.onerror=null; this.src='https://cdn.jsdelivr.net/npm/telesense/dist/tele.umd.min.js';"></script> | |
| onerror="this.onerror=null; this.src='./dist/tele.umd.js';"></script> |
Pull Request: Scroll Bug Fix & Complete UI Modernization
Summary
This PR solves the scroll stuck bug reported in the Telesense project and delivers a comprehensive UI/UX overhaul of the demo page, transforming it from a minimal, functional interface into a modern, gradient-driven, interactive experience.
Key Changes
✨ UI/UX Improvements
🎨 Visual Features
📱 Layout Enhancements
📖 Documentation Section
🔧 Technical
Testing Checklist
Files Modified
demo/index.html- Complete rewrite with new HTML structure and CSSNote: This is a visual and UX enhancement that also fixes the reported scroll bug. No SDK changes were made—only the demo page presentation layer was modernized.