Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add new copy icon #1208

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Add new copy icon #1208

merged 1 commit into from
Apr 12, 2024

Conversation

simon-leech
Copy link
Contributor

@simon-leech simon-leech commented Apr 12, 2024 •
edited

Little PR - I have simply added a new icon to use for copying as I noticed we didn't have one.
Accessed like this for example -

<div class="mask-icon copy"></div>

@simon-leech simon-leech self-assigned this Apr 12, 2024
@simon-leech simon-leech marked this pull request as ready for review April 12, 2024 12:48
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Apr 12, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@RobAndrewHurst RobAndrewHurst merged commit 846be49 into main Apr 12, 2024
5 checks passed
@RobAndrewHurst RobAndrewHurst deleted the icon-copy branch April 12, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RobAndrewHurst RobAndrewHurst RobAndrewHurst approved these changes

@cityremade cityremade Awaiting requested review from cityremade

@AlexanderGeere AlexanderGeere Awaiting requested review from AlexanderGeere

@KieronFerrey KieronFerrey Awaiting requested review from KieronFerrey

Assignees

@simon-leech simon-leech

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@simon-leech @RobAndrewHurst