okuda-seminar / Twitter-Clone

0 stars 0 forks source link

[iOS] Implement Display of LocationPermissionAlertPopUpView in PopUp #447

Closed TakayaShirai closed 1 month ago

TakayaShirai commented 1 month ago

Issue Number

https://github.com/okuda-seminar/Twitter-Clone/issues/442

Implementation Summary

This PR introduces a pop-up display for LocationAccessExplanationPopUpView (previously LocationPermissionAlertPopUpView) utilizing a dimming view. This enhancement aims to improve the user experience within the app.

Scope of Impact

The new pop-up will be triggered when users tap the "Add location" button while creating a new post.

Particular points to check

I have renamed LocationPermissionAlertPopUpView to LocationAccessExplanationPopUpView and refactored the view into three subviews using @ViewBuilder. Although each subview didn't require a lot of code, these naming and structure are intended to enhance comprehension of the view's functionality and hierarchy. Please review if this approach is appropriate.

Reference

https://github.com/user-attachments/assets/b3f4169d-6e35-491f-9ab3-39beec029f3d

Schedule

Until 10/12.