patternfly / patternfly/patternfly-org

Add a page-loaded class to all org pages (not just full-page examples)

Open
#4,616 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

animation PF Team
Dominant language
JavaScript
Stars
74
Forks
147
Avg merge
4d 11h
Merged PRs (30d)
2

Description

Currently, a page-loaded class is added to full-page examples, but not to all pages on the PatternFly org site. For consistency and to enable styling or scripting that depends on page load state, add the page-loaded class to the body (or appropriate container) on all org pages after they finish loading. Note: this should probably be a pf- class now.

This will provide an indication that the page has loaded so that animations/transitions can use the no-motion durations and prevent the appearance of animations happening while the page is loading.

Reference: The implementation already exists for full-page examples.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the existing implementation that adds page-loaded to full-page examples and trace how org pages finish loading. Extend that behavior to the body or appropriate container for all org pages, using a pf- class if appropriate. Done means every org page exposes the class after loading without introducing load-time animations or transitions.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.