bellatinyfish

Bella

@bellatinyfish
GitHub Profile
collaborative and methodical
Highly visual and detailed reviewer who extensively uses screenshots to document progress and issues. Focuses heavily on UI/UX implementation with meticulous attention to design consistency and component functionality.
238
Comments
68
PRs
7
Repos
164
Avg Chars
2
Harshness

Personality

Visual communicator Detail-oriented Collaborative and responsive Design-focused Problem-solving oriented Documentation-heavy Self-reflective about code issues Progress-oriented

Greatest Hits

"somehow all the padding spacing don't work"
"I can make adjustments based on this as well ( spacing and everything )"
"The close button should be consistent across all components"
"merged with changes from main, updated with the latest db.json and API, resolved any conflict"
"looks good"
"moved to this PR"

Focus Areas

Common Phrases

"added" "updated" "moved to this PR" "just" "well" "component" "with overriding" "without overriding" "looks good" "spacing" "consistent across all components" "I can make adjustments" "somehow" "merged with changes from main" "resolved any conflict"

Sentiment Breakdown

neutral
189
positive
19
questioning
7
constructive
15
harsh_questioning
1

Review Outcomes

APPROVED
17
COMMENTED
6
DISMISSED
1

Most Reviewed Authors

bellatinyfish
207
mingyang-tinyfish
9
zifanwTF
6
mmigdol
5
KateZhang98
4
thakkerurvish
3
aarontantf
2
paveldudka
2

Spiciest Comments

agentql/#22 [view]
Looks like this is using MUI 5, but we are using MUI4. Tried to find a way to implement something similar to this using MUI4, but seems like it doesn't work with MUI 4 ?

AI Persona Prompt

You are bellatinyfish, a meticulous UI/UX focused code reviewer who believes in the power of visual documentation. Your reviews are characterized by extensive use of screenshots to illustrate every point, change, and issue. You have a collaborative tone and always include detailed visual evidence of your work. Key behaviors: - Always include screenshots when discussing UI changes, using specific alt text like 'Screenshot 2024-03-15 at 2 38 10 PM' - Use phrases like 'somehow', 'well', 'looks good', 'I can make adjustments', and 'consistent across all components' - Focus heavily on Material-UI theming, spacing, padding, and component consistency - Frequently move work between PRs and reference other PRs with full URLs - Document progress with detailed markdown formatting using headers and numbered lists - Address styling issues methodically, often mentioning 'overriding' vs 'without overriding' - Always mention when you've 'merged with changes from main' and 'resolved any conflict' - Include Storybook integration as part of component completion - Use casual problem-solving language like 'trying to solve it first' and 'I'll fix PRs' - Provide multiple visual examples showing before/after states - Reference design consistency frequently and suggest improvements based on existing patterns Your reviews should be helpful, visual-heavy, and focused on UI implementation details while maintaining a collaborative and solution-oriented approach.

Recent Comments (231 total)

aquarium-old/#120 Added email login [view]
### email and password verification added on both Firebase, and aqua repo. ### here is screenshot of Firebase's record of new email-registered user: <img width="800" alt="Screenshot 2024-05-06 at 4 42 43 AM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/49627568-8af3-4f56-86c3-b80b98697d91"> **next** add display name and image for email registered users **questions** -
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook · src/components/tinyfish/UsageSidebar.js [view]
Of course. This is the Paper component 's output, I can make adjustments based on this as well ( spacing and everything ) <img width="501" alt="Screenshot 2024-03-15 at 2 38 10 PM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/cb1247d8-3f64-4824-84b5-80e7268671a5">
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook · src/components/tinyfish/UsageSidebar.js [view]
done
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook · src/components/tinyfish/UsageSidebar.js [view]
adjusted ``` const percentage = props.percentage; ```
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook · src/components/tinyfish/UsageSidebar.js [view]
After I changed to Paper component , somehow all the padding spacing don't work. Changed parameters and tried different ways in both "useStyles" and "sx" prop, but it just doesn't affect the output in Paper component sx prop <img width="461" alt="Screenshot 2024-03-15 at 3 11 18 PM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/555906e9-d07b-4dcb-ac59-1338eb22accf">
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook · src/components/tinyfish/UsageSidebar.js [view]
I see !!!
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook [view]
Converting to [draft], awaiting backend update
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook [view]
If we are not waiting for date picker to be merged before we merge this one, I think we are good to go.
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook [view]
### Re-created the UI , same with design, merged with changes from main, updated with the latest db.json and API, resolved any conflict. <img width="477" alt="Screenshot 2024-03-25 at 4 51 07 PM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/4d2224d0-fdb4-436b-a3d1-6d1827d9b16f">
aquarium-old/#52 [Usage Sidebar] Billing cycle outlook [view]
Moved to this PR: https://github.com/tinyfish-io/aquarium/pull/63
aquarium-old/#108 Create first api key · src/components/tinyfish/CreateFirstApiKeySection.js [view]
using `vm`, percentage of viewpoint width.
aquarium-old/#108 Create first api key [view]
Also, this is just the component, if the component looks good, I'll go ahead and add the condition for rendering this component.
aquarium-old/#108 Create first api key [view]
image added
aquarium-old/#108 Create first api key [view]
image added <img width="400" alt="Screenshot 2024-04-29 at 6 34 55 PM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/d3293f28-30f3-4940-a3f4-42fa8b06b905">
aquarium-old/#112 Generic Error UI and logic · public/grey-lines.svg [view]
not the same. create first key logo is this: <img width="400" alt="Screenshot 2024-05-01 at 2 16 56 PM" src="https://github.com/tinyfish-io/aquarium/assets/143453177/75336965-f0df-4226-a598-25b12a15509c">