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

Tweak: shareable reading list design feedback #3697

Merged
merged 26 commits into from
Nov 30, 2022

Conversation

cooltey
Copy link
Collaborator

@cooltey cooltey commented Nov 21, 2022

@cooltey cooltey added the WIP Work in progress label Nov 21, 2022
@cooltey cooltey added Design review and removed WIP Work in progress labels Nov 22, 2022
@dbrant
Copy link
Member

dbrant commented Nov 23, 2022

@cooltey Are you updating all of the language translations on TranslateWiki? (if not, they will all be overwritten on the next TWN sync.)

@cooltey
Copy link
Collaborator Author

cooltey commented Nov 23, 2022

@cooltey Are you updating all of the language translations on TranslateWiki? (if not, they will all be overwritten on the next TWN sync.)

Ah! You are correct. I have created another PR for the strings #3700 . Please review and merge it and I'll update the strings on translatewiki soon.

@dbrant
Copy link
Member

dbrant commented Nov 28, 2022

@cooltey All of these changes look ok to me -- Feel free to merge and re-release to beta if the design is approved before Wed.

@dbrant dbrant merged commit ba3e440 into main Nov 30, 2022
@dbrant dbrant deleted the shareable-readinglist-tweak-design branch November 30, 2022 19:43
sharvaniharan added a commit that referenced this pull request Dec 2, 2022
* main:
  Update test releated libraries version (#3708)
  Bump versionCode. (#3707)
  Exit dialog when users accidentally quit new topic  (#3674)
  Update the empty states in the talk page (#3676)
  Add missing preview funnel call.
  Tweak: shareable reading list design feedback (#3697)
  Remove search suggestion related code from the app (#3705)
  Remove unnecessary word from a message (#3704)
  Update RxJava to 3.1.5. (#3702)
  Localisation updates from https://translatewiki.net. (#3701)
  Update shareable reading list strings for translation (#3703)
  Update strings for shareable reading list (#3700)
  Split off Reading Lists surveys into two independent surveys. (#3698)
  Localisation updates from https://translatewiki.net.
  Fix: don't check for explicit storage permission in API 33+. (#3693)
  Factor out a string constant. (#3694)

# Conflicts:
#	app/src/main/java/org/wikipedia/readinglist/ReadingListItemView.kt
#	app/src/main/java/org/wikipedia/readinglist/ReadingListsFragment.kt
#	app/src/main/res/layout/item_reading_list.xml
#	app/src/main/res/values-qq/strings.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants