Unify portfolio card layout and action button placement#62
Conversation
- Group skills and area icons side-by-side in footer with '|' separator in Projects, Experience, and Publications cards. - Position link icons in the top-right corner for Projects and Publications cards. - Move 'Cite' button next to other action buttons in the Publications card footer.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Move skills and areas icons to the card header in Project, Publication, and Experience cards.
- Add a horizontal pipe separator ('|') between areas and skills.
- Relocate the Cite button to the bottom-right footer in Publication cards, adjacent to other artifact links.
- Maintain top-right placement for link icons in Projects and Publications.
- Remove redundant IconOnlyLinks component in favor of unified header layout.
- Ensure consistent spacing and typography across all portfolio card headers and footers.
…headers
- Move Areas and Tools icons to the bottom-left footer in Project, Publication, and Experience cards.
- Maintain a horizontal pipe separator ('|') between Area and Skill icons.
- Ensure link icons are positioned in the top-right header for both Project and Publication cards.
- Keep the 'Cite' button and full action links in the bottom-right footer for Publication cards.
- Revert Experience card header to original layout.
- Introduce compact mode for Publication links to support icon-only header placement.
|
1 participant
This PR unifies the layout of portfolio cards (Projects, Experience, Publications) by standardizing how skills, areas, and links are displayed.
Key changes:
src/components/timeline.tsx(Experience), added a|separator between areas and skills in the footer.src/components/project.tsx(Projects), moved the tools (skills) from the header to the footer, grouped them with areas using a|separator, and ensured link icons remain in the top-right header.src/components/publication.tsx(Publications), moved skills from the header to the footer with a|separator, added icon-only links to the top-right header, and moved theCitebutton to the bottom-right next to other action links.These changes improve visual consistency across the portfolio sections and ensure important actions are easily accessible.
PR created automatically by Jules for task 4746108893916171848 started by @amrabed