Skip to content

Commit 639a4d5

Browse files
committed
fix: address Copilot review feedback
- Fix Twitter URL to use @keaboratory for consistency - Add ariaLabel prop to Timeline components for reusability - Pass custom ariaLabel from PublishingFlowSection Signed-off-by: Gauarv Chaudhary <[email protected]>
1 parent 19d43eb commit 639a4d5

File tree

4 files changed

+12
-8
lines changed

4 files changed

+12
-8
lines changed

src/app/page.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export default function Home() {
4343
<path d="M5.042 15.165a2.528 2.528 0 0 1-2.52 2.523A2.528 2.528 0 0 1 0 15.165a2.527 2.527 0 0 1 2.522-2.52h2.52v2.52zM6.313 15.165a2.527 2.527 0 0 1 2.521-2.52 2.527 2.527 0 0 1 2.521 2.52v6.313A2.528 2.528 0 0 1 8.834 24a2.528 2.528 0 0 1-2.521-2.522v-6.313zM8.834 5.042a2.528 2.528 0 0 1-2.521-2.52A2.528 2.528 0 0 1 8.834 0a2.528 2.528 0 0 1 2.521 2.522v2.52H8.834zM8.834 6.313a2.528 2.528 0 0 1 2.521 2.521 2.528 2.528 0 0 1-2.521 2.521H2.522A2.528 2.528 0 0 1 0 8.834a2.528 2.528 0 0 1 2.522-2.521h6.312zM18.956 8.834a2.528 2.528 0 0 1 2.522-2.521A2.528 2.528 0 0 1 24 8.834a2.528 2.528 0 0 1-2.522 2.521h-2.522V8.834zM17.688 8.834a2.528 2.528 0 0 1-2.523 2.521 2.527 2.527 0 0 1-2.52-2.521V2.522A2.527 2.527 0 0 1 15.165 0a2.528 2.528 0 0 1 2.523 2.522v6.312zM15.165 18.956a2.528 2.528 0 0 1 2.523 2.522A2.528 2.528 0 0 1 15.165 24a2.527 2.527 0 0 1-2.52-2.522v-2.522h2.52zM15.165 17.688a2.527 2.527 0 0 1-2.52-2.523 2.526 2.526 0 0 1 2.52-2.52h6.313A2.527 2.527 0 0 1 24 15.165a2.528 2.528 0 0 1-2.522 2.523h-6.313z" />
4444
</svg>
4545
</a>
46-
<a href="https://twitter.com/Keployio" target="_blank" rel="noopener noreferrer" className="text-gray-400 hover:text-[#F89559] transition-colors" aria-label="Twitter">
46+
<a href="https://twitter.com/keaboratory" target="_blank" rel="noopener noreferrer" className="text-gray-400 hover:text-[#F89559] transition-colors" aria-label="Twitter">
4747
<svg className="w-6 h-6" viewBox="0 0 24 24" fill="currentColor">
4848
<path d="M18.244 2.25h3.308l-7.227 8.26 8.502 11.24H16.17l-5.214-6.817L4.99 21.75H1.68l7.73-8.835L1.254 2.25H8.08l4.713 6.231zm-1.161 17.52h1.833L7.084 4.126H5.117z" />
4949
</svg>

src/app/publishing-flow/page.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export default function PublishingFlowPage() {
5959
</svg>
6060
</a>
6161
<a
62-
href="https://twitter.com/Keployio"
62+
href="https://twitter.com/keaboratory"
6363
target="_blank"
6464
rel="noopener noreferrer"
6565
className="text-gray-400 hover:text-[#F89559] transition-colors"

src/components/sections/PublishingFlowSection.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,15 +92,16 @@ export function PublishingFlowSection() {
9292
<div className="mb-20">
9393
{/* Desktop Horizontal Timeline */}
9494
<div className="hidden lg:block">
95-
<HorizontalTimeline steps={publishingSteps} />
95+
<HorizontalTimeline steps={publishingSteps} ariaLabel="Blog publishing workflow steps" />
9696
</div>
9797

9898
{/* Mobile/Tablet Vertical Timeline */}
9999
<div className="lg:hidden max-w-xl mx-auto">
100-
<Timeline steps={publishingSteps} />
100+
<Timeline steps={publishingSteps} ariaLabel="Blog publishing workflow steps" />
101101
</div>
102102
</div>
103103

104+
104105
{/* CTA Section - Keploy Theme with Stars */}
105106
<div className="max-w-4xl mx-auto">
106107
<div

src/components/ui/timeline.tsx

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ export interface TimelineStep {
1313
interface TimelineProps {
1414
steps: TimelineStep[];
1515
className?: string;
16+
ariaLabel?: string;
1617
}
1718

1819
interface TimelineItemProps {
@@ -25,12 +26,12 @@ interface TimelineItemProps {
2526
* Clean, professional Timeline component for displaying step-by-step workflows.
2627
* Uses white theme with orange accents.
2728
*/
28-
export function Timeline({ steps, className }: TimelineProps) {
29+
export function Timeline({ steps, className, ariaLabel = "Workflow steps" }: TimelineProps) {
2930
return (
3031
<div
3132
className={cn("relative", className)}
3233
role="list"
33-
aria-label="Blog publishing workflow steps"
34+
aria-label={ariaLabel}
3435
>
3536
{steps.map((step, index) => (
3637
<TimelineItem
@@ -44,6 +45,7 @@ export function Timeline({ steps, className }: TimelineProps) {
4445
);
4546
}
4647

48+
4749
function TimelineItem({ step, index, isLast }: TimelineItemProps) {
4850
return (
4951
<div
@@ -92,7 +94,7 @@ function TimelineItem({ step, index, isLast }: TimelineItemProps) {
9294
/**
9395
* Horizontal Timeline for desktop - Clean card-based design
9496
*/
95-
export function HorizontalTimeline({ steps, className }: { steps: TimelineStep[], className?: string }) {
97+
export function HorizontalTimeline({ steps, className, ariaLabel = "Workflow steps" }: { steps: TimelineStep[], className?: string, ariaLabel?: string }) {
9698
return (
9799
<div className={cn("relative", className)}>
98100
{/* Connector line */}
@@ -104,8 +106,9 @@ export function HorizontalTimeline({ steps, className }: { steps: TimelineStep[]
104106
<div
105107
className="grid grid-cols-5 gap-4"
106108
role="list"
107-
aria-label="Blog publishing workflow steps"
109+
aria-label={ariaLabel}
108110
>
111+
109112
{steps.map((step, index) => (
110113
<div
111114
key={step.id}

0 commit comments

Comments
 (0)