Skip to content

Employee map pineapple icon#14407

Merged
rafaeelaudibert merged 6 commits intomasterfrom
cursor/employee-map-pineapple-icon-1c68
Jan 17, 2026
Merged

Employee map pineapple icon#14407
rafaeelaudibert merged 6 commits intomasterfrom
cursor/employee-map-pineapple-icon-1c68

Conversation

@rafaeelaudibert
Copy link
Copy Markdown
Member

Changes

Adds a toggle to the employee map to display team members' pineapple on pizza preferences. When enabled, an icon (yes, no, or unknown) appears next to each avatar with a descriptive tooltip. The toggle is off by default.

Checklist

  • Words are spelled using American English
  • PostHog product names are in title case. It's "Product Analytics" not "Product analytics". If talking about a category of product, use sentence case e.g. "There are a lot of product analytics tools, but PostHog's Product Analytics is the best"
  • Titles are in sentence case
  • Feature names are in sentence case. It's "Click here to create a trend insight" not "... create a Trend Insight" and so on.
  • Use relative URLs for internal links
  • If I moved a page, I added a redirect in vercel.json

Slack Thread

Open in Cursor Open in Web

- Added toggle switch to show/hide pineapple preference icons
- Updated GraphQL query to fetch pineappleOnPizza data for team members
- Added pineapple sticker icons to Avatar component (left side of avatar)
- Displays appropriate sticker based on preference: yes, no, or unknown

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
@cursor
Copy link
Copy Markdown

cursor Bot commented Jan 15, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
posthog Ready Ready Preview Jan 16, 2026 10:48pm

Request Review

- Added toggle switch in the map view to show/hide pineapple preferences
- Updated ProfileNode type to include pineappleOnPizza field
- Added pineapple indicator icons next to employee avatars on the map:
  - Green with pineapple emoji for those who love it
  - Red with no emoji for those who don't
  - Gray with question mark for undecided
- Toggle appears in top-left corner of the map view

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
- Added 'Group' toggle to enable/disable clustering of people on the map
- Toggle is on by default (showing clusters when zoomed out)
- Placed above the pineapple toggle in the top-left corner
- When disabled, shows individual avatars at all zoom levels

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
@rafaeelaudibert
Copy link
Copy Markdown
Member Author

@cursor expose the option to render the team mini badge in there as well! It should be exclusive with Pinneaple. Also, add some styling to the selector along the lines of "Render badge" or just "badge" to separate it from the group option.

Also rename "group" to "clustering" in the UI, it will look much better

@rafaeelaudibert rafaeelaudibert marked this pull request as ready for review January 16, 2026 15:05
@cursor
Copy link
Copy Markdown

cursor Bot commented Jan 16, 2026

Taking a look!

Open in Cursor Open in Web

- Renamed 'Group' toggle to 'Clustering' for clarity
- Added 'Small Team' badge option showing team mini crests
- Refactored badge creation into separate function (createBadgeElement)
  that accepts badge type and profile data
- Badge options: None, Pineapple, Team (with mini crest icons)
- Added GraphQL query for team mini crest data
- Updated UI with badge type selector buttons

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
Copy link
Copy Markdown
Member Author

@cursor when a badge is selected the text is white rather than keeping the gray text

image.png

You should take inspiration from the List / Map selector we already have to understand how to make this work great

image.png

@cursor
Copy link
Copy Markdown

cursor Bot commented Jan 16, 2026

Taking a look!

Open in Cursor Open in Web

Copy link
Copy Markdown
Member Author

@cursor do we really need the changes on Avatar/index.tsx? It looks like you're handling everything on PeopleMap.tsx. Remove that if it's not needed, please.

@cursor
Copy link
Copy Markdown

cursor Bot commented Jan 16, 2026

Fixed the badge selector styling to match the List/Map toggle behavior:

  • Removed text-white from selected badge buttons - the text now stays gray (text-primary) even when a badge is selected
  • Added font-medium for consistent text weight matching the ToggleGroup component
  • Changed background colors from gray scale (bg-gray-100, bg-gray-800) to the design system colors (bg-primary, bg-accent) for proper light/dark theme integration

The badge selector now behaves like the List/Map toggle - only the background color changes when an option is selected, while the text remains the same gray color.

Open in Cursor Open in Web

cursoragent and others added 2 commits January 16, 2026 22:25
- Remove text-white from selected badge buttons
- Use text-primary font-medium for consistent gray text color
- Use bg-primary/bg-accent for proper theme integration
- Matches the styling behavior of the ToggleGroup component

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
- Reverted Avatar/index.tsx pineapple badge additions
- Reverted AboutTeam/index.tsx toggle additions
- The pineapple/team badge feature is only needed in PeopleMap.tsx

Co-authored-by: rafaeelaudibert <rafaeelaudibert@gmail.com>
@rafaeelaudibert rafaeelaudibert merged commit c6ae89c into master Jan 17, 2026
15 checks passed
@rafaeelaudibert rafaeelaudibert deleted the cursor/employee-map-pineapple-icon-1c68 branch January 17, 2026 15:05
sarahxsanders pushed a commit that referenced this pull request Jan 26, 2026
## Changes

Adds a toggle to the employee map to display team members' pineapple on pizza preferences. When enabled, an icon (yes, no, or unknown) appears next to each avatar with a descriptive tooltip. The toggle is off by default.

## Checklist

- [ ] Words are spelled using American English
- [ ] PostHog product names are in [title case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/title-case). It's "Product Analytics" not "Product analytics". If talking about a category of product, use sentence case e.g. "There are a lot of product analytics tools, but PostHog's Product Analytics is the best"
- [ ] Titles are in [sentence case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/sentence-case)
- [ ] Feature names are in **[sentence case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/sentence-case)**. It's "Click here to create a trend insight" not "... create a Trend Insight" and so on.
- [ ] Use relative URLs for internal links
- [ ] If I moved a page, I added a redirect in `vercel.json`

---
[Slack Thread](https://posthog.slack.com/archives/D0A8UV4HBNF/p1768512645178749?thread_ts=1768512645.178749&cid=D0A8UV4HBNF)

<a href="https://cursor.com/background-agent?bcId=bc-f6dd47a6-67c0-4de8-96db-d5a7706c2fc8"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-f6dd47a6-67c0-4de8-96db-d5a7706c2fc8"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>
sarahxsanders pushed a commit that referenced this pull request Feb 27, 2026
## Changes

Adds a toggle to the employee map to display team members' pineapple on pizza preferences. When enabled, an icon (yes, no, or unknown) appears next to each avatar with a descriptive tooltip. The toggle is off by default.

## Checklist

- [ ] Words are spelled using American English
- [ ] PostHog product names are in [title case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/title-case). It's "Product Analytics" not "Product analytics". If talking about a category of product, use sentence case e.g. "There are a lot of product analytics tools, but PostHog's Product Analytics is the best"
- [ ] Titles are in [sentence case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/sentence-case)
- [ ] Feature names are in **[sentence case](https://apastyle.apa.org/style-grammar-guidelines/capitalization/sentence-case)**. It's "Click here to create a trend insight" not "... create a Trend Insight" and so on.
- [ ] Use relative URLs for internal links
- [ ] If I moved a page, I added a redirect in `vercel.json`

---
[Slack Thread](https://posthog.slack.com/archives/D0A8UV4HBNF/p1768512645178749?thread_ts=1768512645.178749&cid=D0A8UV4HBNF)

<a href="https://cursor.com/background-agent?bcId=bc-f6dd47a6-67c0-4de8-96db-d5a7706c2fc8"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-f6dd47a6-67c0-4de8-96db-d5a7706c2fc8"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>
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.

2 participants