Add AgentLab landing page and project pages #303
Conversation
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review by Korbit AI
Korbit automatically attempts to detect when you fix issues in new commits.
| Category | Issue | Status |
|---|---|---|
 |
Invalid HTML attribute in heading element ▹ view | |
 |
Uncached DOM query in click handler ▹ view | |
 |
Fix heading class syntax ▹ view | |
|
Improve meta description ▹ view | |
|
Malformed HTML heading attributes ▹ view | |
|
Inconsistent heading structure ▹ view | |
 |
Incorrect heading class syntax ▹ view | |
|
Redundant HTML comments ▹ view | |
|
Generic Meta Description ▹ view | |
|
Redundant Column Nesting ▹ view |
Files scanned
| File Path | Reviewed |
|---|---|
| docs/landing_page/static/js/index.js | ✅ |
| docs/landing_page/projects/browsergym.html | ✅ |
| docs/landing_page/projects/workarena-plus-plus.html | ✅ |
| docs/landing_page/index.html | ✅ |
| docs/landing_page/projects/focus_agent.html | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Check out our docs on how you can make Korbit work best for you and your team.
| <div class="column is-half"> | ||
| <div class="card"> | ||
| <div class="card-content"> | ||
| <h3 title is-4">Automated Trace Generation</h3> |
There was a problem hiding this comment.
Invalid HTML attribute in heading element 
Tell me more
What is the issue?
The h3 element has an invalid attribute 'title' instead of the correct 'class' attribute.
Why this matters
This will cause the heading to not receive proper CSS styling from the Bulma framework, resulting in incorrect visual appearance and potentially breaking the page layout.
Suggested change ∙ Feature Preview
Replace the invalid 'title' attribute with the correct 'class' attribute:
<h3 class="title is-4">Automated Trace Generation</h3>Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <div class="column is-half"> | ||
| <div class="card"> | ||
| <div class="card-content"> | ||
| <h3 title is-4">Automated Trace Generation</h3> |
There was a problem hiding this comment.
Malformed HTML heading attributes 
Tell me more
What is the issue?
Invalid HTML attribute 'title' without a value and missing 'class=' for the 'is-4' attribute.
Why this matters
This malformed HTML will cause the browser to fail parsing the element correctly, potentially breaking the page layout and CSS styling for this heading.
Suggested change ∙ Feature Preview
Fix the HTML syntax by adding the missing 'class=' attribute:
<h3 class="title is-4">Automated Trace Generation</h3>Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| journal={arXiv preprint arXiv:2407.05291}, | ||
| year={2024} | ||
| }</code></pre> | ||
| <button class="button is-small" onclick="navigator.clipboard.writeText(document.querySelector('pre code').textContent)"> |
There was a problem hiding this comment.
Uncached DOM query in click handler
Tell me more
What is the issue?
Inefficient DOM query using querySelector on every button click without caching the result.
Why this matters
The DOM query executes on every click, causing unnecessary performance overhead when users repeatedly copy the BibTeX citation.
Suggested change ∙ Feature Preview
Cache the DOM element reference or use a more efficient approach:
<button class="button is-small" onclick="copyBibTeX(this)">With a JavaScript function that finds the nearest pre code element relative to the button.
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <div class="column is-half"> | ||
| <div class="card"> | ||
| <div class="card-content"> | ||
| <h3 title is-4">Automated Trace Generation</h3> |
There was a problem hiding this comment.
Fix heading class syntax 
Tell me more
What is the issue?
HTML syntax error in heading tag - incorrect attribute format for 'title' class
Why this matters
The malformed heading affects document structure and accessibility for screen readers
Suggested change ∙ Feature Preview
Automated Trace Generation
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <html> | ||
| <head> | ||
| <meta charset="utf-8"> | ||
| <meta name="description" content="WorkArena++: Towards Compositional Planning and Reasoning-based Common Knowledge Work Tasks"> |
There was a problem hiding this comment.
Improve meta description
Tell me more
What is the issue?
Meta description is too generic and matches the title, missing key information about the benchmark's purpose and scope
Why this matters
Poor meta descriptions reduce SEO effectiveness and make it harder for users to understand the page's unique value
Suggested change ∙ Feature Preview
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <div class="card-content"> | ||
| <h3 title is-4">Automated Trace Generation</h3> |
There was a problem hiding this comment.
Inconsistent heading structure
Tell me more
What is the issue?
HTML attribute syntax error in heading tag - missing class="title" and malformed quotes
Why this matters
Inconsistent heading structure breaks the design pattern used throughout the document and could cause styling issues
Suggested change ∙ Feature Preview
Fix the heading tag to match the pattern used in other cards:
<h3 class="title is-4">Automated Trace Generation</h3>Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <div class="column is-half"> | ||
| <div class="card"> | ||
| <div class="card-content"> | ||
| <h3 title is-4">Automated Trace Generation</h3> |
There was a problem hiding this comment.
Incorrect heading class syntax 
Tell me more
What is the issue?
Missing class attribute syntax in heading tag - 'title' should be 'class="title"'
Why this matters
The incorrect syntax makes the heading styling inconsistent with other headings in the document and could cause styling issues
Suggested change ∙ Feature Preview
Automated Trace Generation
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <!-- PDF Link. --> | ||
| <span class="link-block"> | ||
| <a href="https://arxiv.org/abs/2407.05291" | ||
| class="external-link button is-normal is-rounded is-dark"> |
There was a problem hiding this comment.
Redundant HTML comments
Tell me more
What is the issue?
Unnecessary HTML comments that don't provide additional context beyond what's clear from the code
Why this matters
Comments that merely restate what's obvious from the code add visual noise and make the code harder to scan
Suggested change ∙ Feature Preview
Remove redundant comments and keep only those that provide meaningful context or explanation
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <html> | ||
| <head> | ||
| <meta charset="utf-8"> | ||
| <meta name="description" content="BrowserGym Ecosystem: A unified environment for web agent research across multiple benchmarks."> |
There was a problem hiding this comment.
Generic Meta Description
Tell me more
What is the issue?
The meta description is too generic and doesn't highlight the key differentiators or value proposition of BrowserGym.
Why this matters
A more specific meta description would improve SEO and help users quickly understand BrowserGym's unique benefits.
Suggested change ∙ Feature Preview
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
| <div class="column has-text-centered"> | ||
| <div class="publication-links"> |
There was a problem hiding this comment.
Redundant Column Nesting
Tell me more
What is the issue?
The publication links section is unnecessarily nested within another centered column div, creating redundant structure since it's already inside a centered column.
Why this matters
Extra nesting increases DOM complexity and makes the code harder to maintain without providing any additional layout benefits.
Suggested change ∙ Feature Preview
Remove the extra column div and keep just the publication-links div:
<div class="publication-links">
<!-- Links content -->
</div>Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
|
I updated the colors to meet servicenow color chart: 
|
amanjaiswal73892
force-pushed
the
add-project-pages
branch
from
5590e4a to
84ac0b1
Compare
amanjaiswal73892
left a comment
amanjaiswal73892
left a comment
There was a problem hiding this comment.
LGTM! I fixed WA dead link and table heading text-color with dark bg in the landing page.
2 participants
This PR adds a landing page to AgentLab along with project pages for research papers of ServiceNow Research.
Description by Korbit AI
What change is being made?
Add a complete AgentLab landing page and project pages (BrowserGym Ecosystem, WorkArena++, FocusAgent) with a new GitHub Pages deployment workflow and supporting site assets (index.html, project pages, README, and basic JS/CSS).
Why are these changes being made?
To publish and automatically deploy a public, navigable landing site and project showcase for AgentLab, enabling easy access to documentation, papers, code, and benchmark details. The approach provides a ready-to-run static site template with project pages and a single-source deployment workflow.