Issue 107 committee page frontend fixes#110
Merged
laurenpudz merged 7 commits intomainfrom Feb 18, 2026
Merged
Conversation
- Increased the font size of the committee portrait labels from 0.5rem to sm (0.875rem = 14px for my device) - Increased committee photo and frame sizes to accommodate new font size. The width and height of the frames are now a nice number of 56 in tailwind = 14rem. The ratios of photo width: frame width (0.6) and photo height: frame height (0.6444..) have been maintained with this adjustment, rounded to 1 decimal place (9rem for photo height and 8.4rem for photo width) - Gave the committee section a bottom padding of 20 (tailwind), so there is now equal padding on the bottom and top - Updated MemberSerializer to now include the Member Object's primary key (integer id), denoted as pk. This is necessary for make committee portraits link to their respective member pages - Updated all instances in the codebase where a Member object if fetched, adding the pk attribute to match the updated MemberSerializer - Added conditional rendering logic in the about page: each committee member's photo and name will render as wrapped in a Link to their respective member page, only if the fetched Member object does not have the placeholder pk value of 0 (don't worry, the member id's start at 1, so 0 is never valid), otherwise it will render the photo and name without Link wrappers
- Removed repetition of the same Image component for each conditional render for the committee photos. This was done be simply turning the Image into a function called committeeImage(profilePic: string)
laurenpudz
requested changes
Feb 16, 2026
- Reworked how the committee portraits are listed, such that they are all in one list and not separated into two rows
- Fixed Image components to hopefully prevent warping, by setting the width and height props equal to the tailwinds-set height and width
- Changed frame-photo dimension ratios, may need a future change
- Replaced height and width props for the about section image with fill={true}, so the whole image space is actually filled
- Statically imported landing_placeholder.png to be used for some elements
- Refactored out the code for a whole committee portrait into a single function to be used in the mapping
- Undid a change that is part of another pr (change name referencing a specific variable)
- Undid importing landing_placeholder.png statically, as the type is not accounted for in the project's config.
laurenpudz
approved these changes
Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change Summary
-Increased the font size of the committee portrait labels to md
Change Form
Other Information
Related issue