BOUNTY: Web GUI reprogramming

c0urier

My exact version is:
2.1-RC0 (amd64)
built on Thu May 23 20:43:32 EDT 2013
FreeBSD 8.3-RELEASE-p8

Clear-Pixel

@clear-pixel Don't think you read the last few posts before posting. I solved that problem of storing the column number a few posts back.

I hear what your saying about the GUI. I think the way to go is to get all the css and as much js as we can out of the HTML/PHP pages and into .css & .js files. Then we can work on stuff like collapsible menus and fancy stuff. But until that is done, it would be a nightmare.

Do you think, or do you know you fixed it?

charlie0440

~~Well it works fine on my setup but appears not to be working for c0urier. Perhaps you could test it too?~~

Edit - Yes it works

stephenw10

Maybe a 32bit/64bit issue? Is there anything compiled in there?

It works fine for me on a 32bit nano install, a few ~~days~~ weeks old though:

2.1-BETA1 (i386)
built on Wed May 15 13:19:07 EDT 2013

Looks good. :)

Steve

charlie0440

Thanks for testing Stephen.

~~Its odd it's not working for c0urier as the only changes are a few lines to the index.php and some added Jquery to a javascript file.

@c0urier I pm'd you~~

c0urier

The index.php file was located in /usr/local/www/themes for some reason.
But I can't explain why it ended there as it asked to overwrite index.php.

It works now, thanks!

charlie0440

Glad to here it! ;)

For anyone else trying this please make sure when copying you copy the files to the correct location you need to be copying from the root of the zip files ie if you extracted the zip to a folder called tmp you need to cd into tmp
cd /location/of/folder/tmp/
scp -r * root@192.168.0.50:/usr/local/www/

that will place index.php to /usr/local/www/index.php (overwrite)
and the full_screen folder to /usr/local/www/themes/full_screen

charlie0440

tiny bug fix to bring this inline with my github.

Found a bug a few days ago where it was adding the add/delete column buttons to the picture widget

If your updating it may be easier to make the change yourself (usr/local/www/themes/full_screen/loader.js line 111):

change

jQuery('form')

to:

jQuery('form[action="index.php"]')

as seen here: https://github.com/CharlieMarshall/pfsense/commit/0e94111d38c67d56901df3573e8af29afaf767d8

or download the whole theme from:

download link: https://dl.dropboxusercontent.com/u/12367642/full_screen_theme.zip
youtube video: http://youtu.be/irglXyn41Ys
Installation instructions: http://forum.pfsense.org/index.php/topic,48140.msg338715.html#msg338715

Supermule

Also working for 2.0.3?

charlie0440

@Supermule:

Also working for 2.0.3?

Afraid not, jQuery is a requirement which got added in 2.1.

I can try and hack one together for you.

Supermule

That would be awesome!

Then we could have a package for that if possible!

stephenw10

There seem to be some spurious add/delete column buttons on the RRD graph page.

Steve

charlie0440

See about 5 posts above. I think that will fix it. Let me know if not

stephenw10

Hmm, yes I saw that and made the change. Perhaps the old file is cached. I'll look again.
Edit: Yep, fixed. :)

Steve

c0urier

I can't get this working with current RC.
2.1-RC0 (amd64)
built on Wed Jun 12 06:19:17 EDT 2013

I've tried to redownload the package:
cd /usr/local/www
fetch https://dl.dropboxusercontent.com/u/12367642/full_screen_theme.zip
unzip full_screen_theme.zip

index.php source:
http://pastebin.com/0vZbd5qk

charlie0440

that's the original index.html it doesn't look like you have overridden it with the index.php from the zip

Same problem you had before

gogol

@charlie0440:

that's the original index.html it doesn't look like you have overridden it with the index.html from the zip

Same problem you had before

I downloaded the widescreen package zip today from dropbox, but there is no index.php.in it

charlie0440

@gogol this technically isn't the widescreen package, it acts the same but creates new theme called full_screen which is widescreen. If you downloaded from the link listed 2 posts above by c0urier there is definitely a index.php file, inside the zip there is the following:

index.php
themes (folder)

Edit: On another note I see there was a commit merged with master while I was on holiday which made a slight alteration to index.php (charset encoding). I will update the zip tonight to add this change:

c0urier

You might be right, but when I do:
[2.1-RC0][USER@FIREWALL]/usr/local/www(22): unzip full_screen_theme.zip
Archive: full_screen_theme.zip
replace index.php? [y]es, [n]o, [A]ll, [N]one, [r]ename: y
inflating: index.php
replace themes/full_screen/all.css? [y]es, [n]o, [A]ll, [N]one, [r]ename: A

And it's still the same, should it be replacing the original index.php file?

[2.1-RC0][USER@FIREWALL]/usr/local/www(28): rm index.php
Gives me - 404 - Not Found
[2.1-RC0][USER@FIREWALL]/usr/local/www(29): unzip full_screen_theme.zip
Archive: full_screen_theme.zip
inflating: index.php
replace themes/full_screen/all.css? [y]es, [n]o, [A]ll, [N]one, [r]ename: A

Gives me a working site with the following code:
http://pastebin.com/VagW2iYB

gogol

@charlie0440:

@gogol this technically isn't the widescreen package, it acts the same but creates new theme called full_screen which is widescreen. If you downloaded from the link listed 2 posts above by c0urier there is definitely a index.php file, inside the zip there is the following:

index.php
themes (folder)

Edit: On another note I see there was a commit merged with master while I was on holiday which made a slight alteration to index.php (charset encoding). I will update the zip tonight to add this change:
162	 	 -
 	162	 +

165	 	 -  
 	165	 +  

Thanks, maybe something went wrong on my end.
I wonder why your commit #654 still isn't merged.
Your "index.php" is now based on which version of the snapshot? I don't have the latest and I don't want to make a mess. I didn't replace it for now.