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.

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

OJ-3050: fix - include title within the legend #1164

Merged
merged 2 commits into from
Feb 20, 2025
Merged

OJ-3050: fix - include title within the legend #1164

merged 2 commits into from
Feb 20, 2025

Conversation

MarieseMikely
Copy link
Contributor

@MarieseMikely MarieseMikely commented Feb 19, 2025

Proposed changes

What changed

  • Update html tags to use be the same as here
  • Updated h1 to be within the form legend
Screenshot 2025-02-19 at 16 46 25
  • Removed blank <p> tags by updating the string
Screenshot 2025-02-19 at 16 06 02

Why did it change

Issue tracking

Checklists

Environment variables or secrets

  • No environment variables or secrets were added or changed

Other considerations

  • Update README with any new instructions or tasks

@MarieseMikely MarieseMikely force-pushed the OJ-3050 branch 2 times, most recently from af3fb6d to e191038 Compare February 19, 2025 16:09
@MarieseMikely MarieseMikely marked this pull request as ready for review February 19, 2025 16:12
@MarieseMikely MarieseMikely requested review from a team as code owners February 19, 2025 16:12
@MarieseMikely MarieseMikely changed the title OJ-3050: fix - include title within the legend and removed duplicate … OJ-3050: fix - include title within the legend Feb 19, 2025
@MarieseMikely MarieseMikely merged commit 5b97654 into main Feb 20, 2025
9 checks passed
@MarieseMikely MarieseMikely deleted the OJ-3050 branch February 20, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants