Skip to content

Feature/accordion cards #294

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
Draft
wants to merge 40 commits into
base: feature/interaction-panel
Choose a base branch
from
Draft

Feature/accordion cards #294

wants to merge 40 commits into from
+677 −42

Conversation

SaqAsh
Copy link
Contributor

@SaqAsh SaqAsh commented Jun 25, 2025

Summary

Issues

  • #(issue number)
  • #(issue number)

Description of Changes

Readiness Checklist

  • Self Review
    • I have performed a self review of code
    • I have run the application locally and manually tested the feature
    • I have checked all updates to correct typos and misspellings
  • Formatting
    • Code follows the project style guide
    • Autmated code formatters (ie. Prettier) have been run
  • Local Testing
    • Successfully built all packages locally
    • Successfully ran all test suites, all unit and integration tests pass
  • Updated Tests
    • Unit and integration tests have been added that describe the bug that was fixed or the features that were added
  • Documentation
    • All new environment variables added to .env.schema file and documented in the README
    • All changes to server HTTP endpoints have open-api documentation
    • All new functions exported from their module have TSDoc comment documentation

SaqAsh added 30 commits June 10, 2025 10:59
@SaqAsh
initial component initialization
89ed33f
@SaqAsh
allow the ability to actually initalize the openInit
30f6bee
@SaqAsh
add line seperation, fix up styling
694931f
@SaqAsh
make it so that we have appropriate aligning
dc2949b
@SaqAsh
Merge branch 'feature/header-component' into feature/accordian-cards
d372d65 
merge from pr review
@SaqAsh
move the state management inside of the accordion, pt1
6bfcc69
@SaqAsh
whitespace
322c21d
@SaqAsh
hover state
0d98f89
@SaqAsh
Merge branch 'feature/expand-collapse-all-button' into feature/accord…
c1f2083 
…ian-cards
@SaqAsh
add back disasbled state from interaction panel
634107b
@SaqAsh
generated dictionarypage for now just to see everything working at on…
659a456 
…ce, need to figure out how we want to display this properly later.
@SaqAsh
instead of increasing the border width, add a box shadow instead
b0b0dab
@SaqAsh
make hover state less prominent
4b42736
@SaqAsh
add a new button variant, iconOnly
140eaf6
@SaqAsh
styling buttons such that we have the ability to pass in custom style…
229928f 
…s for the accordion
@SaqAsh
add the hash
6bb1355
@SaqAsh
fix some div hierarchy issues with the spans not properly being stack…
9b344ed 
… and an uneccessary div
@SaqAsh
remove errors from stories, need to consider the case for a longer ti…
7de835c 
…tle accordion
@SaqAsh
attempt 1 of getting the title to wrap when too big
7516587
@SaqAsh
implement handling of a long title
1997c6c
@SaqAsh
refactor the read-more/read-less logic
316750b
@SaqAsh
make the transition smooth with css without the use of useEffect
7744643
@SaqAsh
clean up css finally
50503e6
@SaqAsh
whitespace fix
b2bc3eb
@SaqAsh
general copying pattern
2da8a34
@SaqAsh
finalize the url hash navigation
0c6f710
@SaqAsh
rework the table of contents dropdown stuff, and chat gpt the diction…
5973dd4 
…ary page to work again with new props
@SaqAsh
add a todo for the buttons rendering
69467f0
@SaqAsh
Merge branch 'feature/interaction-panel' into feature/accordian-cards
a7edf37
@SaqAsh
data dictionary page was a mess and not needed currently
06fe036
@SaqAsh SaqAsh changed the base branch from main to feature/interaction-panel June 25, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SaqAsh