Unlock all styles now that Sustainer is discontinued - #1847
Conversation
Generated by 🚫 Danger |
|
📲 You can test the changes from this Pull Request in Simplenote Android by scanning the QR code below to install the corresponding build.
|
|
@mzorz thoughts on this? since sustainer is done away with, maybe we release this to all? |
There was a problem hiding this comment.
Pull request overview
Removes the now-defunct “premium” gating from style selection so all users can choose and apply any available style (including the Black/AMOLED style) across the app UI and widgets.
Changes:
- Always apply the user-selected style in
ThemeUtils.getStyle()and widget/layout selectors (no fallback to Default for non-premium). - Simplify
StyleActivityUX by removing lock indicators and the locked/upsell dialog so tapping a style always selects it. - Update Settings’ style summary to always reflect the selected style name, and add release notes entry for the change.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Simplenote/src/main/java/com/automattic/simplenote/utils/ThemeUtils.java | Always returns the selected style (removes premium gate). |
| Simplenote/src/main/java/com/automattic/simplenote/utils/PrefUtils.java | Removes premium-based fallbacks for widget layouts and dialog theme selection; removes unused default style name helper. |
| Simplenote/src/main/java/com/automattic/simplenote/StyleActivity.java | Removes premium lock UI/upsell dialog; enables selecting any style. |
| Simplenote/src/main/java/com/automattic/simplenote/PreferencesFragment.java | Settings summary always shows the currently selected style. |
| RELEASE-NOTES.txt | Documents unlocking styles for all users. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| public StyleAdapter(List<String> styles) { | ||
| this.mStyles = new ArrayList<>(styles); | ||
| mSelectedPosition = mIsPremium ? PrefUtils.getStyleIndexSelected(StyleActivity.this) : STYLE_DEFAULT; | ||
| mSelectedPosition = PrefUtils.getStyleIndexSelected(StyleActivity.this); | ||
| } |
| notifyItemChanged(mSelectedPosition); | ||
| mSelectedPosition = holder.getAdapterPosition(); | ||
| notifyItemChanged(mSelectedPosition); | ||
| PrefUtils.setStyleIndex(StyleActivity.this, holder.getAdapterPosition()); |
|
You don't need all these changes to enable styles. Just update |
Thank you. will do |
12d22e5 to
6e28103
Compare
Fix
Simplenote Sustainer has been discontinued, and
pref_key_premiumis never written anywhere in the codebase's history, soisPremium()was alwaysfalse— leaving every style except Default permanently locked and the Style row hidden in Settings, even though a pure-black AMOLED style (Black) already exists.Per review feedback, this is the minimal change:
PrefUtils.isPremiumnow defaults totrue(no device ever stored a value, so this unlocks everyone).pref_key_stylerow inpreferences.xmlis visible again (app:isPreferenceVisible="true").Fixes #1771
Fixes #1661
Test
Review
Anyone can review. Note: with
isPremium()defaulting to true, the (still hidden) membership row would show "premium" if ever un-hidden.Release
RELEASE-NOTES.txtwas updated in 6e28103 with: