Skip to content

Conversation

@rmathew1011
Copy link

@rmathew1011 rmathew1011 commented Jan 22, 2026

Description

This feature enables the expand collapse feature in the research areas in an Organization.

  • added #mainContent template reference to main-content div section
  • Added a listener block for registering click events to toggle research areas - expand or collapse
  • updated aria-expanded and button label - Show more and Show less on toggle.

Type of change

  • New feature (non-breaking change which adds functionality)

Testing Procedures

Verified this feature works on local.
Click link: To see more click on Show more link.
Screenshot 2026-01-22 at 5 46 26 PM
Click link: To see the collapsed version, click Show less link.
Screenshot 2026-01-22 at 5 46 46 PM

@rmathew1011 rmathew1011 changed the base branch from tamu-main to sprint-17-staging January 22, 2026 23:50
@rmathew1011 rmathew1011 deleted the test-section-expand-collapse branch January 23, 2026 17:29
@rmathew1011 rmathew1011 restored the test-section-expand-collapse branch January 23, 2026 23:07
@rmathew1011 rmathew1011 reopened this Jan 26, 2026
@rmathew1011 rmathew1011 requested a review from a team January 26, 2026 20:06
Copy link

@kaladay kaladay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please refactor to remove out WIP commits like:

WIP - show more button clicked successfully code clean up needed

 - Code clean up per PR suggestions.

   Link: #522
event.preventDefault();
event.stopPropagation();

const container = toggleBtn.closest('.research-areas') as HTMLElement | null;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've looked over this closer and I think this type to null is not necessary and should be removed because of the if (!container) return; after that.

The | null should be removed.

const container = toggleBtn.closest('.research-areas') as HTMLElement;

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.

In Organization Profile View: reduce the space taken by "Research Areas"

3 participants