Closed jamestagal closed 2 years ago
@jamestagal, thanks for raising this. I have uncovered the issue. I'll work on pushing a fix out.
Hi @jamestagal, please pull the latest theme. I made a big push with commits that fix the bug you highlighted, plus some other cleanup, refactor to make it more DRY, and refinements to the theme
Hi Roberto,
thanks for letting me know. I run a plenti theme update comdendium
which updated my theme folder to the latest hash commit cbc6ceed503735ad364937cca5a66a24c49987bc
However I noticed now the white text for part of the site name and nav is now black so I can't see them until I hover. See screenshot.
Did you change the text colour for the colourScheme for Technobeat? i am a little confused here because I had copied over the themes CSS into the Styles folder in my project so I thought i would be still using my version.
Also I noticed that the svelte files that I have modified in my project (outside of the theme folder) will not reflect the changes you just made...is there an easy way to merge those changes into my project? I haven't committed your recent change to my repo so the changes are just in my project locally.
Hi Benjamin,
Super cool that updating the theme is so seamless. Looks like you are on the correct version. I did not change the css files. I did, however, make significant changes to components in layouts/
. Your fix was 1 of 20 commits that I made today. There were a bunch of other little fixes, cleanups, and enhancements.
Because we are using git, my recommendation would be to copy over each of the components that you customized from the themes folder into your project layouts/
folder. I'm assuming that you are using VSCode or similar to edit your code. If so, you can click on the source control icon to get the list of uncommitted changes. Then you will be able to click on each file to see the diffs between your version and my updated file. Then you will be able to incorporate your changes or revert the file altogether as desired.
That said, there are four commits that you should look at. One of which is the likely culprit:
layouts/content/index.svelte
bug fix for "Frame"layouts/style/global.svelte
bug fix -- this one contains two class renames which impact:
layouts/global/
footer.svelte -- rename class from footer to footnotelayouts/global/
navbar.svelte -- rename class from menu to navmenu --> your likely white font challengeThe remaining commits contain changes for:
The class renames are part of a broader theme refactor that I am working on for a more robust, general, and easier to implement template.
BTW... you could revert the class renames and just incorporate the bug fixes. This would give you no refactor now for a potentially bigger refactor tomorrow, if you want to stay in sync with the theme in the future. ;-)
Hi @roobyz Thanks for all this information. I am in the process of incorporating your changes to my modified versions. All seems to be going well.
I have a question regarding the this commit: Sort projects posts by project
https://github.com/plenti-themes/compendium/commit/321dfad2fa83571811c9c18a11b46549bd3cc1e6
// Sort project posts in ascending order
allProjs.sort(function(a, b) {
// var s = new Date(b.fields.dateModified).getTime() - new Date(a.fields.dateModified).getTime();
var p = a.fields.project.localeCompare(b.fields.project)
return p == 0? a.path.localeCompare(b.path) : p;
});
I am guessing this code is sorting based on alphabetical order? Because I saw that change in my code.
Cheers, Ben
hi @jamestagal,
Yeah, I find that JS is a bit obtuse at times. The p variable is a comparison between the project names. If the posts belong to the same project (p==0) then it returns the comparison between the post names to sort them alphabetically within a project, else it sorts the projects alphabetically.
Have a look at my project page to see how this works in practice. Which by the way, goes into my thinking around how I plan to streamline the theming.
I believe that ideal state would be the projects get sorted chronologically, and the posts alphabetically within a project. I haven't figured out how to accomplish this, yet. Feedback, welcomed. ;-)
Hi @roobyz That's great..I like your site and the text styles and colours etc..Nice! I have modified my code to show Projects as Posts because I don't think I would have projects per se but individual posts.. https://vigorous-einstein-b90548.netlify.app/posts/ But I like that I can name similar posts/projects and group them around similar topics. And then Categories and Tags further help find content. For instance I have Posts on Speaking Events and Learning Vietnamese so I have grouped them accordingly. Thanks for all your work on this. It tremendous. BTW. @jimafisk helped me implement sorting posts chronologically from posts (dateCreated) see issue below. https://github.com/jamestagal/edtechdesigner/issues/13 Hope that helps, Regards, Ben
Hi @jamestagal
Thanks for the feedback! Much appreciated. ;-)
I like how your repurposed "Projects". Very creative. It shows that my design is flexible, too. Yeah @jimafisk has provided stellar support and a great project!
Would you say that this issue is resolved now?
Cheers, Roberto
Love the collab guys! @roobyz thanks for taking a look at these issues, I'm super impressed with all the fixes / improvements you've pushed over the last couple of days! @jamestagal nice job sticking with the subtheming. your site is coming out awesome!
Thanks again for all your work @roobyz and @jimafisk . I appreciate the support and your project.
sort
BTW... thanks for this insight. I remember now that Jim had provided me that sort module and I had forgotten it. I'm going to implement it, as well. I think it will make for much more readable code. I'll create a new function for sort_by_post, so that folks can easily swap. ;-)
Cheers, Roberto
Hi Jim, I just tried to have a look at the "Frame"; one of the featured page display options but got an error. Both the "Bleed" and "None" option work fine. Pls see screenshot below.
Maybe this issues is related to this issue @roobyz