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

[Gutenberg] - Remove tooltip flag and modal layout feature flag #13665

Merged
merged 9 commits into from
Jan 13, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion WordPress/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ android {
buildConfigField "long", "REMOTE_CONFIG_FETCH_INTERVAL", "10"
buildConfigField "boolean", "FEATURE_ANNOUNCEMENT_AVAILABLE", "false"
buildConfigField "boolean", "GUTENBERG_MENTIONS", "true"
buildConfigField "boolean", "MODAL_LAYOUT_PICKER", "true"
buildConfigField "boolean", "HOME_PAGE_PICKER", "true"
buildConfigField "boolean", "UNIFIED_LOGIN_AVAILABLE", "true"
buildConfigField "boolean", "WP_STORIES_AVAILABLE", "true"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,6 @@
import org.wordpress.android.util.WPActivityUtils;
import org.wordpress.android.util.analytics.AnalyticsUtils;
import org.wordpress.android.util.analytics.service.InstallationReferrerServiceStarter;
import org.wordpress.android.util.config.ModalLayoutPickerFeatureConfig;
import org.wordpress.android.util.config.MySiteImprovementsFeatureConfig;
import org.wordpress.android.viewmodel.main.WPMainActivityViewModel;
import org.wordpress.android.viewmodel.mlp.ModalLayoutPickerViewModel;
Expand Down Expand Up @@ -208,7 +207,6 @@ public class WPMainActivity extends LocaleAwareActivity implements
@Inject UploadUtilsWrapper mUploadUtilsWrapper;
@Inject ViewModelProvider.Factory mViewModelFactory;
@Inject PrivateAtomicCookie mPrivateAtomicCookie;
@Inject ModalLayoutPickerFeatureConfig mModalLayoutPickerFeatureConfig;
@Inject ReaderTracker mReaderTracker;
@Inject MediaPickerLauncher mMediaPickerLauncher;
@Inject MySiteImprovementsFeatureConfig mMySiteImprovementsFeatureConfig;
Expand Down Expand Up @@ -441,7 +439,7 @@ private void initViewModel() {
handleNewPostAction(PagePostCreationSourcesDetail.POST_FROM_MY_SITE);
break;
case CREATE_NEW_PAGE:
if (mModalLayoutPickerFeatureConfig.isEnabled() && mMLPViewModel.canShowModalLayoutPicker()) {
if (mMLPViewModel.canShowModalLayoutPicker()) {
mMLPViewModel.createPageFlowTriggered();
} else {
handleNewPageAction("", "", null,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ import org.wordpress.android.util.DisplayUtils
import org.wordpress.android.util.QuickStartUtils
import org.wordpress.android.util.ToastUtils.Duration
import org.wordpress.android.util.WPSwipeToRefreshHelper
import org.wordpress.android.util.config.ModalLayoutPickerFeatureConfig
import org.wordpress.android.util.helpers.SwipeToRefreshHelper
import org.wordpress.android.util.redirectContextClickToLongPressListener
import org.wordpress.android.util.setLiftOnScrollTargetViewIdAndRequestLayout
Expand Down Expand Up @@ -99,7 +98,6 @@ class PagesFragment : Fragment(), ScrollableViewInitializedListener {
@Inject lateinit var progressDialogHelper: ProgressDialogHelper
@Inject lateinit var uploadActionUseCase: UploadActionUseCase
@Inject lateinit var uploadUtilsWrapper: UploadUtilsWrapper
@Inject lateinit var modalLayoutPickerFeatureConfig: ModalLayoutPickerFeatureConfig

private var quickStartEvent: QuickStartEvent? = null
private var progressDialog: ProgressDialog? = null
Expand Down Expand Up @@ -347,7 +345,7 @@ class PagesFragment : Fragment(), ScrollableViewInitializedListener {
})

viewModel.createNewPage.observe(viewLifecycleOwner, Observer {
if (modalLayoutPickerFeatureConfig.isEnabled() && mlpViewModel.canShowModalLayoutPicker()) {
if (mlpViewModel.canShowModalLayoutPicker()) {
mlpViewModel.createPageFlowTriggered()
} else {
createNewPage()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,6 @@
import org.wordpress.android.util.analytics.AnalyticsUtils.BlockEditorEnabledSource;
import org.wordpress.android.util.config.ConsolidatedMediaPickerFeatureConfig;
import org.wordpress.android.util.config.GutenbergMentionsFeatureConfig;
import org.wordpress.android.util.config.ModalLayoutPickerFeatureConfig;
import org.wordpress.android.util.config.TenorFeatureConfig;
import org.wordpress.android.util.config.WPStoriesFeatureConfig;
import org.wordpress.android.util.helpers.MediaFile;
Expand Down Expand Up @@ -404,7 +403,6 @@ enum RestartEditorOptions {
@Inject TenorFeatureConfig mTenorFeatureConfig;
@Inject GutenbergMentionsFeatureConfig mGutenbergMentionsFeatureConfig;
@Inject XPostsCapabilityChecker mXpostsCapabilityChecker;
@Inject ModalLayoutPickerFeatureConfig mModalLayoutPickerFeatureConfig;
@Inject ConsolidatedMediaPickerFeatureConfig mConsolidatedMediaPickerFeatureConfig;
@Inject CrashLogging mCrashLogging;
@Inject MediaPickerLauncher mMediaPickerLauncher;
Expand Down Expand Up @@ -688,8 +686,7 @@ protected void onCreate(Bundle savedInstanceState) {
private void presentNewPageNoticeIfNeeded() {
if (mIsPreview
|| !mIsPage
|| !mIsNewPost
|| !mModalLayoutPickerFeatureConfig.isEnabled()) {
|| !mIsNewPost) {
return;
}
String message = mEditPostRepository.getContent().isEmpty() ? getString(R.string.mlp_notice_blank_page_created)
Expand Down Expand Up @@ -2348,7 +2345,6 @@ private GutenbergPropsBuilder getGutenbergPropsBuilder() {
isUnsupportedBlockEditorEnabled,
unsupportedBlockEditorSwitch,
mIsPreview,
mModalLayoutPickerFeatureConfig.isEnabled(),
wpcomLocaleSlug,
postType,
themeBundle
Expand Down Expand Up @@ -3298,14 +3294,6 @@ private void showSuggestions(SuggestionType type, Consumer<String> onResult) {
ActivityLauncher.viewSuggestionsForResult(this, mSite, type);
}

@Override public void onGutenbergEditorSetStarterPageTemplatesTooltipShown(boolean tooltipShown) {
AppPrefs.setGutenbergStarterPageTemplatesTooltipShown(tooltipShown);
}

@Override public boolean onGutenbergEditorRequestStarterPageTemplatesTooltipShown() {
return AppPrefs.getGutenbergStarterPageTemplatesTooltipShown();
}

@Override
public void onHtmlModeToggledInToolbar() {
toggleHtmlModeOnMenu();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,6 @@ public enum DeletablePrefKey implements PrefKey {
SHOULD_AUTO_ENABLE_GUTENBERG_FOR_THE_NEW_POSTS,
SHOULD_AUTO_ENABLE_GUTENBERG_FOR_THE_NEW_POSTS_PHASE_2,
GUTENBERG_OPT_IN_DIALOG_SHOWN,
GUTENBERG_STARTER_PAGE_TEMPLATES_TOOLTIP_SHOWN,

IS_QUICK_START_NOTICE_REQUIRED,
LAST_SKIPPED_QUICK_START_TASK,
Expand Down Expand Up @@ -887,14 +886,6 @@ public static void removeSupportName() {
remove(DeletablePrefKey.SUPPORT_NAME);
}

public static void setGutenbergStarterPageTemplatesTooltipShown(boolean tooltipShown) {
setBoolean(DeletablePrefKey.GUTENBERG_STARTER_PAGE_TEMPLATES_TOOLTIP_SHOWN, tooltipShown);
}

public static boolean getGutenbergStarterPageTemplatesTooltipShown() {
return getBoolean(DeletablePrefKey.GUTENBERG_STARTER_PAGE_TEMPLATES_TOOLTIP_SHOWN, false);
}

/*
* returns a list of local IDs of sites recently chosen in the site picker
*/
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,6 @@ public interface EditorFragmentListener extends DialogVisibilityProvider {
void onGutenbergEditorSessionTemplatePreviewTracked(String template);
void showUserSuggestions(Consumer<String> onResult);
void showXpostSuggestions(Consumer<String> onResult);
void onGutenbergEditorSetStarterPageTemplatesTooltipShown(boolean tooltipShown);
boolean onGutenbergEditorRequestStarterPageTemplatesTooltipShown();
String getErrorMessageFromMedia(int mediaId);
void showJetpackSettings();
void onStoryComposerLoadRequested(ArrayList<Object> mediaFiles, String blockId);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnLogGutenbergUserEventListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnReattachMediaSavingQueryListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnReattachMediaUploadQueryListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnStarterPageTemplatesTooltipShownEventListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnMediaEditorListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnMediaLibraryButtonListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnMediaFilesCollectionBasedBlockEditorListener;
Expand Down Expand Up @@ -69,7 +68,6 @@ public void attachToContainer(ViewGroup viewGroup, OnMediaLibraryButtonListener
OnGutenbergDidSendButtonPressedActionListener
onGutenbergDidSendButtonPressedActionListener,
ShowSuggestionsUtil showSuggestionsUtil,
OnStarterPageTemplatesTooltipShownEventListener onSPTTooltipShownEventListener,
OnMediaFilesCollectionBasedBlockEditorListener
onMediaFilesCollectionBasedBlockEditorListener,
boolean isDarkMode) {
Expand All @@ -88,7 +86,6 @@ public void attachToContainer(ViewGroup viewGroup, OnMediaLibraryButtonListener
onGutenbergDidRequestUnsupportedBlockFallbackListener,
onGutenbergDidSendButtonPressedActionListener,
showSuggestionsUtil,
onSPTTooltipShownEventListener,
onMediaFilesCollectionBasedBlockEditorListener,
isDarkMode);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnLogGutenbergUserEventListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnReattachMediaSavingQueryListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnReattachMediaUploadQueryListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnStarterPageTemplatesTooltipShownEventListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnMediaLibraryButtonListener;
import org.wordpress.mobile.WPAndroidGlue.WPAndroidGlueCode.OnMediaFilesCollectionBasedBlockEditorListener;

Expand Down Expand Up @@ -367,6 +366,7 @@ public void gutenbergDidSendButtonPressedAction(String buttonType) {
mEditorFragmentListener.showJetpackSettings();
}
},

new ShowSuggestionsUtil() {
@Override public void showUserSuggestions(Consumer<String> onResult) {
mEditorFragmentListener.showUserSuggestions(onResult);
Expand All @@ -376,17 +376,7 @@ public void gutenbergDidSendButtonPressedAction(String buttonType) {
mEditorFragmentListener.showXpostSuggestions(onResult);
}
},
new OnStarterPageTemplatesTooltipShownEventListener() {
@Override
public void onSetStarterPageTemplatesTooltipShown(boolean tooltipShown) {
mEditorFragmentListener.onGutenbergEditorSetStarterPageTemplatesTooltipShown(tooltipShown);
}

@Override
public boolean onRequestStarterPageTemplatesTooltipShown() {
return mEditorFragmentListener.onGutenbergEditorRequestStarterPageTemplatesTooltipShown();
}
},
new OnMediaFilesCollectionBasedBlockEditorListener() {
@Override public void onRequestMediaFilesEditorLoad(ArrayList<Object> mediaFiles, String blockId) {
mEditorFragmentListener.onStoryComposerLoadRequested(mediaFiles, blockId);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ data class GutenbergPropsBuilder(
private val enableUnsupportedBlockEditor: Boolean,
private val unsupportedBlockEditorSwitch: Boolean,
private val enablePreviewMode: Boolean,
private val enableModalLayoutPicker: Boolean,
private val localeSlug: String,
private val postType: String,
private val editorTheme: Bundle?
Expand All @@ -31,7 +30,6 @@ data class GutenbergPropsBuilder(
translations = GutenbergUtils.getTranslations(activity),
isDarkMode = GutenbergUtils.isDarkMode(activity),
htmlModeEnabled = isHtmlModeEnabled,
isPreview = enablePreviewMode,
isModalLayoutPickerEnabled = enableModalLayoutPicker
isPreview = enablePreviewMode
)
}
2 changes: 1 addition & 1 deletion libs/gutenberg-mobile
Submodule gutenberg-mobile updated 62 files
+15 −3 .circleci/config.yml
+13 −6 .eslintrc.js
+1 −1 .github/PULL_REQUEST_TEMPLATE.md
+5 −0 RELEASE-NOTES.txt
+7 −0 bin/ci-checks-js.sh
+4 −84 bundle/android/strings.xml
+2,721 −2,747 bundle/ios/App.js
+1 −1 bundle/ios/App.js.map
+4 −41 bundle/ios/GutenbergNativeTranslations.swift
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ar.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/bg.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/bo.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ca.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/cs.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/cy.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/da.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/de.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/el.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-au.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-ca.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-gb.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-nz.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/en-za.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/es-ar.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/es-cl.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/es-cr.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/es.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/fa.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/fr.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/gl.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/he.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/hr.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/hu.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/id.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/is.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/it.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ja.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ka.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ko.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/nb.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/nl-be.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/nl.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/pl.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/pt-br.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/pt.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ro.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ru.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sk.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sq.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sr.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/sv.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/th.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/tr.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/uk.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/ur.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/vi.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/zh-cn.json
+1 −1 bundle/ios/assets/gutenberg/packages/react-native-editor/i18n-cache/data/zh-tw.json
+1 −1 gutenberg
+4,718 −4,177 package-lock.json
+3 −5 package.json
+1 −0 src/test/index.js