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

Feature branch: Recommended Content #4859

Merged
merged 122 commits into from
Sep 5, 2024
Merged

Feature branch: Recommended Content #4859

merged 122 commits into from
Sep 5, 2024

Conversation

cooltey
Copy link
Collaborator

@cooltey cooltey commented Aug 6, 2024

@cooltey cooltey added the WIP Work in progress label Aug 6, 2024
@cooltey cooltey requested a review from dbrant as a code owner August 6, 2024 23:39
@cooltey cooltey requested a review from dbrant August 31, 2024 02:40
@cooltey cooltey removed the WIP Work in progress label Aug 31, 2024
@cooltey
Copy link
Collaborator Author

cooltey commented Aug 31, 2024

@dbrant All comments were addressed. This is ready for review again.

app/src/main/java/org/wikipedia/history/HistoryFragment.kt Outdated Show resolved Hide resolved
app/src/main/res/values/styles.xml Outdated Show resolved Hide resolved
app/src/main/res/values/styles.xml Show resolved Hide resolved
app/src/main/res/values/styles.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
dbrant and others added 2 commits September 4, 2024 09:11
…4916)

* Use style instead of deprecated method for resizing dialog window.

* Create child style.

* Automatic parent.

* Whoops.
@cooltey cooltey added Ready to merge PR passed design signoff and ready to be merged. and removed Design review labels Sep 5, 2024
@cooltey cooltey requested a review from dbrant September 5, 2024 17:40
@dbrant
Copy link
Member

dbrant commented Sep 5, 2024

🎉

@dbrant dbrant merged commit 4a3f0af into main Sep 5, 2024
2 checks passed
@dbrant dbrant deleted the recommended-content-design branch September 5, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to merge PR passed design signoff and ready to be merged.
Development

Successfully merging this pull request may close these issues.

3 participants