Flarum Global Color Changer

Frink

Razor I have updated it. Is it better?

Frink

@Shahiem I merged your PR with my repo. Thanks!

Frink

HELP!! ? I am trying to add a settings modal to my extension, but it isn't showing up. See GIT repo link above. What am I doing wrong? Is the namespace wrong? Is it not reading my en.yml file right?

I tried changing the namespaces to the extension id listed in the database, but still no luck...

Frink

I discovered that I hadn't put a link to the extension.js in the admin directory in the bootstrap.php file, so I did that, but that also didn't resolve it....

Shahiem

Frink No problem ? I'll check out your extension later.

Frink

Shahiem OK, Thanks!

Frink

I have updated the git repo with the suggestions that @clarkwinkelmann made to me on discord, but the settings modal button is still not showing up (wow, that was a lot of fancy shift + 8 to get bold and italics ? ). Anybody have any tips on what might be causing this?

Frink

I have also made an AddClientAssets.php file (see git link https://github.com/ZapTechAdmin/flarum-global-color-changer/blob/master/src/listener/AddClientAssets.php), but what should I use for the namespace? I tried ZapTech\Flarum-Global-Color-Changer\Listeners, but my php debugger had a fit! I also tried ZapTech\Global\Color\Changer\Listeners, and that also didn't work... @luceos is there something really stupid in here that I am not noticing, like last time?

Ralkage

Frink you want to follow the psr-4 autoload namespace you provided in your composer.json as so namespace ZapTech\Global-Color-Changer\listener;.

May I also suggest that you change the listener directory to Listeners in the even you need to add more listeners in the future? Also, if you have not already noticed from popular extensions, you should always capitalize and camelcase (when applicable) your namespaces (PHP class files included) as well as with your folders just to keep up with the standards.

Tidy up your composer.json as well.

Frink

Ralkage OK, I will attempt this tomorrow. Sorry about the coding and composer.json, this is my first time actually coding a proper extension, so it will be messy, I will tidy it up as I get better.

0E800

@Frink

Way to stick with it. I am anxious to test it out.

Frink

OK, I have ran into another problem. I made an AddClientAssets listener, and called that from my bootstrap.php. Now, I am just getting a blank page. Nothing in the logs, debug mode is on, nothing in the console either. I pushed my latest changes to the git repo, so if there are any errors, you would be able to tell in their. Is there something that isn't right?

Frink

Fixed! #THEAWESOME @clarkwinkelmann got it fixed!

Frink

EXTENSION UPDATE:

It still does nothing. No big surprise, this is my first extension. But, I have gotten it to do this:

Use an addclientassets.php file instead of a bootstrap.php
Created a settings modal (this was like, an insane moment for me)
Created a textbox on the settings modal for you to enter the hex code into

Other than that, it does absolutely nothing. It is still a major WIP. If you want to assist with development, post on this topic, and I will add you as a contributor to the git repo.

Frink

This is probably a really stupid thing to post an update for, but there is now a text field below the hex code field explaining what that hex code does. If this is a stupid idea, and not necessary, just let me know below and I will remove it.

Frink

Hey @clarkwinkelmann the computer I was using is being used by someone else so I will be offline till the morning. Sorry for the abrupt end!

Razor

It's a good thing to have a text which explain what hex color will change. Maybe, to add separate fields, for example bg, navbar etc, if it's not already there.
Also to add color picker after a box field, to make selecting color easier.

Frink

Razor I added the help text. I also set up the local system with translations. Did you clone the git repo to your local computer?

I am planning to add separate fields for different things once I get the first field working ? but until then, it will just be the hex code.

Thanks for your feedback, it is much appreciated!

Razor

Nope, i didn't. As i do not have flarum installed on localhost/vps. Looking forward to next update.
Want to know, will this be aviable for all users to use or just admins?