Skip to content

Conversation

@danilobatson
Copy link
Contributor

Closes #7909

Summary

This PR refactors the BlueSpacing component by extracting it from BlueComponents.js into its own dedicated file (BlueSpacing.tsx).

Changes

  • Removed BlueSpacing and alike components from BlueComponents.js
  • Created a new BlueSpacing.tsx file
  • In the spirit of keeping the commit small. Imports BlueSpacing into BlueComponents.js where it is then exported for relevant files

Type of Change

  • Refactor (code change that does not affect external behavior)

Checklist

  • Code compiles without errors
  • All tests pass

@danilobatson danilobatson requested a review from limpbrains May 30, 2025 18:50
Copy link
Collaborator

@limpbrains limpbrains left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thank you!

@danilobatson
Copy link
Contributor Author

@marcosrdz @limpbrains @Overtorment

Thanks for the approval! As this would be my first merge here with you all, what would be next steps here as I am not authorized to merge?

@danilobatson
Copy link
Contributor Author

danilobatson commented Jun 2, 2025

@limpbrains @marcosrdz can I get an assist on getting this merged? Have the two review approvals.

@Overtorment
Copy link
Member

oops, needs conflicts resolved, otherwise lgtm

@danilobatson
Copy link
Contributor Author

oops, needs conflicts resolved, otherwise lgtm

@Overtorment just resolved the conflicts. Thank you

@Overtorment Overtorment merged commit def6191 into BlueWallet:master Jun 18, 2025
7 checks passed
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.

Move BlueSpacing from BlueComponents.js to Its Own File

4 participants