Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Something else, for discussion, the theme panel, name column
If a site has only one (1) theme installed, should the “Assign sections” link and the “in use” note be actually displayed? Or should these be suppressed in this case? It looks kinda weird now, and a bit pointless.
And I don’t think a Section can exist without a theme assigned.
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Ah, I see. <txp:header value="application/xml; charset=utf-8" /> is missing name attribute, it must be
<txp:header name="Content-Type" value="application/xml; charset=utf-8" />I will revert name to 4.7-default Content-Type, sorry for the hassle.
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
phiw13 wrote #320685:
If a site has only one (1) theme installed, should the “Assign sections” link and the “in use” note be actually displayed?… I don’t think a Section can exist without a theme assigned.
True at the moment. But see this thread on article snippets for a possible future in which sections may be themeless. Then marking them up this way makes sense.
But if we decide not to pursue this way of working then maybe yes we could hide the pills and (perhaps) link. Will give it some thought.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
I’m curious about the 502 errors, I’ll investigate that – thanks for the heads up.
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
colak wrote #320681:
Accessing beta-demo.textpattern.co/index.php?s=sitemap returns a 502 error.
I can replicate this. Here’s the error in the Nginx log:
2019/12/24 13:31:55 [error] 14090#14090: *9004 upstream sent invalid status "application/xml; charset=utf-8" while reading response header from upstream, client: [IPv4 address], server: beta-demo.textpattern.co, request: "GET /index.php?s=sitemap HTTP/2.0", upstream: "fastcgi://unix:/var/run/php/php7.4-fpm.sock:", host: "beta-demo.textpattern.co"Investigating…
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
etc wrote #320689:
Ah, I see.
<txp:header value="application/xml; charset=utf-8" />is missingnameattribute, it must be […]
Confirming this fixes the 502 on the beta site.
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
with the name attribute, all is working just fine.
Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Yes adding the name attribute makes all working fine. I didn’t know you had changed that.
And OK I see you reverted that requirement. That is probably better for backwards compatibility. I’ll still make it a habit to add the name attribute.
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg
Offline
#33 2019-12-24 23:09:19
- Myusername
- Member
- Registered: 2019-12-12
- Posts: 165
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Bloke wrote #320690:
True at the moment. But see this thread on article snippets for a possible future in which sections may be themeless. Then marking them up this way makes sense.
But if we decide not to pursue this way of working then maybe yes we could hide the pills and (perhaps) link. Will give it some thought.
I’ve now read partly through that thread, with some difficult but interesting concepts. In case you do introduce and use themeless sections, then yes, it probably makes sense, given how it now works, to keep the link (assign sections) next to the theme name. Not sure I fully understand those themeless sections ATM, but will keep reading.
The “pill” note (“in use”) might still be redundant, though ?
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
phiw13 on Codeberg
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
phiw13 wrote #320701:
Not sure I fully understand those themeless sections ATM, but will keep reading.
Nothing’s decided yet but it’s just a notion whereby you could house unpublished content that you can call into other articles. Such sections could act as silos for content snippets – events, profiles, whatever – that you don’t have to relegate to forms (which are presentation not content) nor have to risk having a discoverable URL.
If a section has no theme, it has no output, therefore the content wouldn’t appear on your site or navigation unless you explicitly ask your templates to grab it as part of other articles in other sections.
The “pill” note (“in use”) might still be redundant, though ?
Yes. We could perhaps hide this if there’s only one theme installed.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Feedback to: Textpattern CMS 4.8.0-beta released
Bloke wrote #320707:
We could perhaps hide this [in-use pill] if there’s only one theme installed.
Done.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline