Comments (4)
I can't reproduce this. In fact, the "show more" button does indeed reveal extra CSS revisions for me when I follow your instructions. However, I discovered the following two problems in this area related to JetPack_Public_CSS::revisions_meta_box:
- The call to wp_revisions_to_keep on line 891 produces a PHP notice because $safecss_post is an array, but the function assumes an object. The solution is to either convert to an object before passing (ew) or to no longer use wp_revisions_to_keep here. I chose the latter option.
- The "Show more" link appears even if there are no additional revisions to show. That's a bit misleading so it should probably go away when all the revisions (up to max) are listed.
I fixed both of these issues, but it's unclear to me if/how these are related to the above behavior ... especially hard to say since I can't reproduce it.
Anyway patch to follow -- I hope this helps.
from jetpack.
@jeherve I am also unable to reproduce this. Maybe @mattoperry solved it with the other commit?
from jetpack.
The issue (and the notice) disappeared after applying your patch. Closing this issue. Thank you!
from jetpack.
No worries -- happy to take more issues -- just let me know.
from jetpack.
Related Issues (20)
- AI Featured Image: add video entry on the product page, on My Jetpack
- Tiled Gallery block does not respect the `Circle` setting on live pages HOT 2
- AI Featured Image: include the featured image generation as a feature on the Jetpack AI pricing table
- Boost: Pricing tables out of sync
- AI Featured Image: track error messages on event related to errors
- AI Assistant: Add error messages to AI Control on inline extensions
- Donations Block: Introduce a cap/limit to donations HOT 1
- Boost: Make it possible to switch between WPSC and Boost Cache without disabling them
- Make the sharing buttons that show up on posts/pages an FSE block HOT 2
- Newsletter Dashboard: Wireframes/MVP HOT 1
- AI Assistant: Allow extension-specific prompts
- AI Assistant: Fix Try Again behavior on inline extensions
- PHP Warning/Fatal on jetpack-unrelated pages HOT 2
- Boost: Finalize LIAR
- Boost: Update pricing table to mention LIAR
- My Jetpack: Typo for Connection Flow HOT 1
- AI Assistant: Add events to inline extensions
- AI Assistant: Implement Heading inline extension
- wp_is_stream() causing strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated HOT 2
- Enhancement: Ensure SSO Tooltips are accessible
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from jetpack.