Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AddCategoryFragment.java to AddCategoryFragment.kt #20980

Merged

Conversation

neeldoshii
Copy link
Contributor

@neeldoshii neeldoshii commented Jun 13, 2024

Description

This pull request aims to convert AddCategoryFragment.java to AddCategoryFragment.kt.

Fixes/Related

Screenshots/ Video

addcategorykt.mp4

Steps to reproduce

  1. Create a Post
  2. Click on Any of your Posts
  3. Go to Post Settings
  4. Click on Categories
  5. On top right click on Add Button (+ Button) for adding new categories
  6. CollapseFullScreenDialogFragment is reproduced 🥇

PR Submission Checklist:

  • I have completed the Regression Notes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Testing Checklist (strike-out the not-applying and unnecessary ones):

  • WordPress.com sites and self-hosted Jetpack sites.
  • Portrait and landscape orientations.
  • Light and dark modes.
  • Fonts: Larger, smaller and bold text.
  • High contrast.
  • Talkback.
  • Languages with large words or with letters/accents not frequently used in English.
  • Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
  • Large and small screen sizes. (Tablet and smaller phones)
  • Multi-tasking: Split screen and Pop-up view. (Android 10 or higher)

@neeldoshii neeldoshii changed the title UI enhancement add category fragment AddCategoryFragment.java to AddCategoryFragment.kt Jun 13, 2024
Copy link

sonarcloud bot commented Jun 13, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@neeldoshii
Copy link
Contributor Author

neeldoshii commented Jun 13, 2024

@irfano -- Ready for review.

@irfano irfano self-requested a review June 17, 2024 12:51
Copy link
Member

@irfano irfano left a comment

Choose a reason for hiding this comment

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

Thank you for helping to increase Kotlin usage in the repository. I reviewed the PR and added some comments for you.

}
}

@Suppress("Deprecation")
Copy link
Member

Choose a reason for hiding this comment

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

We shouldn't add any new @Suppress when migrating to Kotlin. You can use Bundle.getSerializableCompat() for getSerializable(). Check usages of it in other Kotlin classes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in the latest commit 👍

Copy link
Member

@irfano irfano left a comment

Choose a reason for hiding this comment

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

LGTM! 👍🏻 Thank you for addressing my comments and contributing to the repository.

@irfano irfano merged commit 666334c into wordpress-mobile:trunk Jun 20, 2024
21 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.

2 participants