refactor: simplify error page layout - #144
Conversation
… and updating styles
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR simplifies the error page by removing the animated glitch heading and button container, replacing them with a single static text element and corresponding minimal CSS, and bumps the error stylesheet version reference. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="templates/error.html" line_range="30-31" />
<code_context>
- </div>
+ <div class="error-text">{{ error_code }}, {{ error_message }}</div>
</div>
- <div class="container">
- <div class="btn"><a href="/"><pre>Home</pre></a></div>
- </div>
-{% endblock %}
</code_context>
<issue_to_address>
**question:** Removing the Home button may impact navigation from error pages.
This change removes a clear recovery path for users who hit this error (e.g., via deep links without visible navigation). If the main layout doesn’t already provide an obvious way back, consider keeping a small link to the homepage or a help page here.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| <div class="container"> | ||
| <div class="btn"><a href="/"><pre>Home</pre></a></div> |
There was a problem hiding this comment.
question: Removing the Home button may impact navigation from error pages.
This change removes a clear recovery path for users who hit this error (e.g., via deep links without visible navigation). If the main layout doesn’t already provide an obvious way back, consider keeping a small link to the homepage or a help page here.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 14 minutes and 55 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Refactors the HTML/CSS for the browser error page to remove the “glitch” effect and simplify the layout/styling.
Changes:
- Simplified
templates/error.htmlmarkup by removing the glitch wrapper and the Home button container. - Removed the corresponding button + glitch animation CSS and introduced a simple
.error-textstyle. - Bumped the cache-busting query param for
error.cssfromv=1tov=2.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
templates/error.html |
Simplifies error page content structure and updates the referenced stylesheet version. |
static/css/error.css |
Removes unused glitch/button styling and adds a simpler .error-text rule. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by Sourcery
Simplify the error page layout and styling by removing decorative effects and custom button container.
Enhancements: