Opened 10 years ago
Closed 10 years ago
#895 closed enhancement (fixed)
Direct links to preview don't trigger the preview overlay
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Theme Directory | Keywords: | |
Cc: |
Description
Links like https://wordpress.org/themes/anchorage/preview/ (the link shown when on Anchorage's Preview) should automatically open a preview of Anchorage instead of showing the theme details page like it does right now
Change History (4)
#2
@
10 years ago
So far I thought about
- redirecting previews to their theme pages and just make previews only accessible through the JS app.
- adding a rewrite rule that would pass on the theme name, show the theme page, and let the JS app pick up on the preview context through the url. We should mitigate duplicate content issues here though.
- adding a rewrite rule that would pass on the theme name and preview context, and we could then create a custom template for that. I'm not sure how that'll work together with the JS app though.
@dd32, do you have a better idea? What would you recommend the best way to go about it would be?
#3
@
10 years ago
- Disable the /preview/ link, and replace it with a wp-themes.com/ link, handle the preview within JS only
- Handle the condition, with a rewrite rule
- Handle the condition within PHP/WordPress with a rewrite rule
Ditch server-side rendering for anything that isn't required, and use the fallbacks where appropriate, in this case, a direct link to wp-themes.com as a non-JS fallback seems correct.
Note: See
TracTickets for help on using
tickets.
That theme has previewception going on. There's a theme made preview on the preview page.