Automattic / Automattic/jetpack

Carousel: update colors of the previous / next buttons

Open
#25,982 0 comments 0 reactions 0 assignees View on GitHub
[Feature] Carousel [Plugin] Jetpack [Pri] Low Enhancement
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

### Impacted plugin

Jetpack

### What

Previous / Next buttons do not follow the same patterns as the other icons in the Carousel view. Let's update them to be consistent.

### How

Previous / Next buttons have 2 states: when not hovered, they're faded out, and they only light up when hovered:

https://user-images.githubusercontent.com/426388/187917688-f63dc57f-a401-4103-9172-7c6ab049b0a3.mov

The other buttons in the interface, however, are all white by default. Maybe the prev/next buttons should too?

Contributor guide

Open the contributing guide

Research direction

Start by locating the Jetpack Carousel implementation and the previous/next button styles, then compare them with the other icons in the Carousel view. The issue's video and description show the current faded-on-idle behavior; done means the buttons follow the agreed consistent color treatment.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.