Skip to content

Comments

Feat: Implement core FAB foundation (PR 1 of 3)#115

Open
Aaratrika-Shelly wants to merge 1 commit intoSrinjoyeeDey:mainfrom
Aaratrika-Shelly:feature/fab-pr1
Open

Feat: Implement core FAB foundation (PR 1 of 3)#115
Aaratrika-Shelly wants to merge 1 commit intoSrinjoyeeDey:mainfrom
Aaratrika-Shelly:feature/fab-pr1

Conversation

@Aaratrika-Shelly
Copy link

@Aaratrika-Shelly Aaratrika-Shelly commented Oct 15, 2025

Description (PR 1 of 3)

This is a new, clean Pull Request for the first phase of the Floating Action Button feature. The original PR, #49 , was closed due to a stale and confusing diff on GitHub.

This is the first of three planned pull requests to implement the new Floating Action Button (FAB) feature, as discussed in the comments.

This foundational PR includes:

  • A new EmergencyFab.tsx component.
  • The core FAB with a functional expand/collapse state (isOpen).
  • A simple vertical menu of the three action buttons.
  • Full compatibility with the project's Dark/Light mode theme.
  • Bonus: Removed an unused api import and function from the Dashboard for better code health.

This PR intentionally does not include animations or tooltips, as those will be handled in PR #2.

Related Issue

Screenshots

Desktop View:
image
image

Mobile View (Expanded Menu):
image
image

@Aaratrika-Shelly
Copy link
Author

Hi @SrinjoyeeDey,
Hope you're having a wonderful week!
Just wanted to gently follow up on this new PR. I'm really excited to move on to the next phase (animations and tooltips) as per our plan.
Please let me know if there's anything else you need from me to get this foundational part merged.
Thank you!"

@SrinjoyeeDey
Copy link
Owner

Yess @Aaratrika-Shelly , I am currently reviewing your PR. will let you know about the rest soon

@Aaratrika-Shelly
Copy link
Author

Hi @SrinjoyeeDey,
Hope you're having a productive week!
I'm just checking in on this PR one last time. With the Hacktoberfest deadline getting closer, I'm really hoping to complete all three planned PRs for this feature before the end of the month.
Of course, I can't start on PR #2 until this foundational part is merged. Is there any feedback I can address to help get this PR ready for merging?
I'm ready to make any necessary changes immediately. Thank you!

@Aaratrika-Shelly
Copy link
Author

HI @SrinjoyeeDey
Are there any changes required from my end to merge the PR?

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