Skip to content
Open

wip #41

Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file added .DS_Store
Binary file not shown.
6 changes: 5 additions & 1 deletion _layouts/default.html
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@
<link rel="stylesheet" href="/assets/components/colomnar-layout.css" />
<link rel="stylesheet" href="/assets/components/default-layout.css" />
<link rel="stylesheet" href="/assets/components/posts.css" />

<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Bricolage+Grotesque:opsz,wght@12..96,200..800&display=swap" rel="stylesheet">
</head>

<body class="dark">
Expand All @@ -57,7 +61,7 @@
href="https://app.managerbot.dev/authentication/register"
class="button secondary"
>
Get Started
Create Account
<img
data-echo="/assets/images/heart.svg"
alt=""
Expand Down
Binary file added assets/images/chart-leadtime@2x.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/images/img-non-linear-mobile@2x.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/images/img-non-linear@2x.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/images/masthead-img@2x.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
74 changes: 74 additions & 0 deletions assets/main.css
Original file line number Diff line number Diff line change
Expand Up @@ -101,3 +101,77 @@ p {
flex-direction: column;
}
}



// <weight>: Use a value from 200 to 800
// <uniquifier>: Use a unique and descriptive class name

.bricolage-grotesque-<uniquifier> {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The class .bricolage-grotesque-<uniquifier> contains placeholders that need to be replaced with actual values before use.

font-family: "Bricolage Grotesque", sans-serif;
font-optical-sizing: auto;
font-weight: <weight>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The <weight> placeholder needs to be replaced with an actual font weight value between 200 and 800.

font-style: normal;
font-variation-settings:
"wdth" 100;
}

.hero {
flex-direction: row !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using !important unless absolutely necessary. It makes CSS harder to maintain and override.

}
.content {
text-align:left;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in text-align:left;. Follow consistent spacing in CSS properties.

max-width:500px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in max-width:500px;. Maintain consistent formatting.

}

.hero .content h1 {
font-size:52px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in font-size:52px;.

margin-bottom:20px !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using !important in margin-bottom:20px !important;.

}

.ht-tagline {
font-size: 22px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in font-size: 22px;.

line-height:1.4;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in line-height:1.4;.

font-weight:300;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in font-weight:300;.

margin-bottom:30px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in margin-bottom:30px; and trailing space at the end of the line.

}
.button {
background-color: #077ACB !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid hardcoding colors like #077ACB. Consider using CSS variables for consistent theming. Also remove !important.

}

.button.secondary {
background-color: var(--royal-900) !important;;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the extra semicolon in var(--royal-900) !important;; and the !important declaration if possible.

}

h2 {
font-size: 32px;
line-height: auto;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line-height: auto; is not a valid value. Use a specific measurement or unitless number.

text-transform: none;
margin-bottom:10px !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in margin-bottom:10px and avoid using !important.

}

p {
font-size: 22px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in font-size: 22px;.

line-height:31.6px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in line-height:31.6px;.

color: #D2C7DD;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider using a CSS variable instead of hardcoding color #D2C7DD.

}

.full-width-layout__content {
margin:0 !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in margin:0 and avoid using !important.

align-items:center;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after colon in align-items:center;.

text-align: center;
}

.old-way .full-width-layout__content p {
max-width: var(--old-way-copy-max-width, 80%) !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using !important in selectors. Consider restructuring CSS to achieve the same result without it.

}

.old-way img {
margin-top: var(--old-way-image-margin, 30px) !important;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using !important in margin-top: var(--old-way-image-margin, 30px) !important;.

}

.text-gradient {
background: -webkit-linear-gradient(0deg, #FEBA01, #F22089);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use standard CSS gradient syntax with vendor prefixes or consider using modern unprefixed gradients with fallbacks.

-webkit-background-clip: text;
-webkit-text-fill-color: transparent;
}
2 changes: 1 addition & 1 deletion assets/pasta/base-variables.css
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
--light-font-weight: 200;

/* Font Family */
--font-family: outfit, sans-serif;
--font-family: "Bricolage Grotesque", sans-serif;

/* Body Fonts */
--body-font-size: 1.2rem;
Expand Down
77 changes: 32 additions & 45 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -3,74 +3,61 @@
description: Improve developer productivity with software engineering intelligence and developer tools
layout: default
---

<section class="hero">
<div class="content">
<h1>Unlock your engineering team’s full potential</h1>
<p>
Getting a pull request reviewed requires manual legwork; selecting a
reviewer, following up multiple times to get anything done, and then an
extensive back and forth. Not only is this a waste of time, but it’s a
black hole, with little to no reporting on where your bottle necks are or
how to improve things.
</p>
<h1>Actionable GitHub Insights for Engineering Teams</h1>
<div class="ht-tagline">
Track your team’s Pull Request lifecycles and optimise their performance with ManagerBot.
</div>
<a
href="https://app.managerbot.dev/authentication/register"
class="button cta"
>
Create Account
<i><img src="assets/images/angle.svg" alt="" /></i>
</a>
</div>
<div class="integrations">
<img
src="/assets/images/github.svg"
alt="Github Hero Image"
class="hero__image"
/>
{% include arrow.html %}
<img
src="/assets/images/mb-mark.svg"
alt="Github Hero Image"
class="hero__image"
/>
{% include arrow.html %}
<div style="width:783px; height:auto;">
<img
src="/assets/images/slack.png"
src="/assets/images/masthead-img@2x.png" style="width:783px; height:auto;"
alt="Github Hero Image"
class="hero__image"
/>
<img
src="/assets/images/icon-discord.svg"
alt="discord icon"
class="hero__mini-icon"
/>
</div>
<div>
<a
href="https://app.managerbot.dev/authentication/register"
class="button cta"
>
Get started for free
<i><img src="assets/images/angle.svg" alt="" /></i>
<div class="stripes"></div>
</a>
</div>
</section>
<section class="full-width-layout old-way">
<section class="full-width-layout old-way" style="background: -webkit-linear-gradient(#090622, #171336); padding-bottom:0;">
<div class="full-width-layout__content">
<h2 class="full-width-layout__heading">The old way of doing things</h2>
<h2>The Non-Linear Reality of the Pull Request Lifecycle</h2>
<p>
Getting a pull request reviewed requires manual legwork; selecting a
reviewer, following up multiple times to get anything done, and then an
extensive back and forth. Not only is this a waste of time, but it's a
black hole, with little to no reporting on where your bottle necks are or
how to improve things.
Developers lose productivity due to context switching from chasing reviewers, working on other tasks, and waiting for approvals.
</p>
</div>
<img
class="desktop"
data-echo="/assets/images/squiggles.svg"
data-echo="/assets/images/img-non-linear@2x.png"
alt="shows inneficiencies of team"
/>
<img
class="mobile"
data-echo="/assets/images/squiggles-mobile.svg"
data-echo="/assets/images/img-non-linear-mobile@2x.png"
alt="shows inneficiencies of team"
/>
</section>
<section class="full-width-layout old-way" style="background-color:#171336">
<div class="full-width-layout__content">
<h2>How ManagerBot Brings Order to the <span class="text-gradient">Chaos</span></h2>
<p>
ManagerBot exposes the bottlenecks and provides engineering leaders with data to remove blockers and improve lead time.
</p>
<img
class="desktop"
data-echo="/assets/images/chart-leadtime@2x.png"
alt="Pull Request Lead Time"
/>
</div>

</section>
<section class="full-width-layout new-way">
<div class="full-width-layout__content">
Expand Down