-
Notifications
You must be signed in to change notification settings - Fork 83
Follow Me: add a button-only mode #1133
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
Conversation
Also make button text and button size configurable
558ecab to
9056a64
Compare
|
@pfefferle Should we merge as-is? I've tried to get the core button working with some AI help, but it's quite a big change where I worry it'll mess with existing blocks. I think the way it is now should work well for the vast majority of situations. We could come back with follow-ups if we wanted to change the block further |
* Follow Me: add a button-only mode Also make button text and button size configurable * proper naming of sidebar panel * changelog * code style * Add changelog item * Update build after merge * Remove changelog changes after merge * Only add left margin to the button when there's profile content. --------- Co-authored-by: Matthias Pfefferle <pfefferle@users.noreply.github.com> Co-authored-by: Konstantin Obenland <obenland@gmx.de>
|
Thanks for doing this. got rid of a few lines of CSS i had hiding the extra stuff. Its nice to see when an old request gets closed with a proper fix. |
* tag '5.7.0': (29 commits) Release 5.7.0 (Automattic#1574) Fix: Show error if site uses "Almost Pretty Permalink" structure (Automattic#1570) Admin: Add padding to extra fields nav links (Automattic#1569) Signature: Add compat for more key encodings (Automattic#1557) Reply block: Improve fallback embed (Automattic#1560) Add label for "Health Check" and "REST API" (Automattic#1571) change changelog items (Automattic#1572) Transformers: Allow setting properties to false. (Automattic#1567) Upgrades: Delete orphaned extra fields (Automattic#1566) Fix: Missing Actor in Outbox-Activities (Automattic#1564) Outbox: Properly handle username requests (Automattic#1559) Import: Load on admin_init (Automattic#1561) Move: support same-server domain migrations (Automattic#1530) Follow Me: add a button-only mode (Automattic#1133) Tests: Convert timestamp to int for comparison (Automattic#1556) Reply: Make Mastodon embeds work (Automattic#1555) Add: `Vary` header settings (Automattic#1552) Actors: Don't convert non-numeric strings to Blog user id (Automattic#1554) Add: Shared Inbox setting (Automattic#1553) Add: New Health checks (Automattic#1524) ...

At long last, you can configure the "Follow Me" button to have a button-only mode.
Fixes #478
Proposed changes:
The sidebar looks like this now:
Other information:
Testing instructions:
Before trying this PR, configure various instances of the Follow Me block. Ensure that they render the same way after this branch is run.
After checking this branch out, try out the new functionality and ensure everything behaves properly.