-
-
Notifications
You must be signed in to change notification settings - Fork 721
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
add logic to show introduction paragraph is presented differently in the editing view #4790
base: main
Are you sure you want to change the base?
Conversation
Hi @rajparad! Apologies for the review delay on this one. This all looks good from a code perspective - nice work! I will ask our designers to review further in case they have any additional behavior or copy tweaks before merging. Thanks! |
No worries, Thanks for the reply!! let me know if there is any feedback from designers :) |
Hello @tonisevener, There was a feedback from design to show alert. there are 2 ways. Let me know what is your preference from code perspective. I can update accordingly :). Design feedback is here: https://phabricator.wikimedia.org/T212318 |
Hi @rajparad - apologies for the delay. I think we can get pretty close to option #1. You'll want to put in in the correct places in
The design won't match up exactly, but that is okay. You can trigger within loadContent once all the content is loaded in this method, if no issues have cropped up (i.e. no blocked error, no edit notice is displayed, etc.). If an edit notice modal is automatically presented, you also want to try triggering it after that edit notice modal is dismissed. Usually the featured article of the day has an edit notice, if you want to test that and see how it behaves. Hope this helps. Let us know if you have any other questions, thanks! |
# Conflicts: # Wikipedia/iOS Native Localizations/en.lproj/Localizable.strings
Hello @tonisevener, Thanks for the feedback. I have updated the code to show warning. For now it's only show one time for a user. Here is the link: https://github.com/wikimedia/wikipedia-ios/pull/4790/files Only thing I am curious is it's now only work with English. I am not how to work with localization in project. Let me know if it's fine or not. |
Phabricator: https://phabricator.wikimedia.org/T212318
Notes
Test Steps
Screenshots/Videos