BOUNTY: Web GUI reprogramming

JeGr

I agree with Stephen. Just dropping a crude one-liner to a good effort on re-designing a theme is very much a pointless thing to do. And referencing a UX blah article, that does nothing to clear up the confusion about your lack of explaining why you dislike the main menu on top (when in fact your link shows many sites that have exactly that) is just - I don't know - senseless? If you want to criticize, feel free, but would it be too much then to actually tell, why you think it is a bad decision in general or subjectively?

I can see both parts. I use quite a few (mobile) devices in portrait mode. A vertical menu bar would cut into the (limited) space of those devices so a top menu is great for those. On a big tablet I can easily go to landscape and a vertical menu would be nice. Dito for the big screen on the wall. So both have pros and cons.

Greets

kelsen

Good afternoon, I've made some changes on metrolab theme, now it's showing alerts in bell icon as shown in screenshot below.

iFloris

@kelsen:

Good afternoon, I've made some changes on metrolab theme, now it's showing alerts in bell icon as shown in screenshot below.

Looks really nice, I look forward to using your theme sometime soon!

kelsen

@iFloris:

@kelsen:

Good afternoon, I've made some changes on metrolab theme, now it's showing alerts in bell icon as shown in screenshot below.

Looks really nice, I look forward to using your theme sometime soon!

I appreciate it, you can download the updated theme from here http://www.4shared.com/archive/C_KrFRy8/metrolabtar.html

kelsen

I appreciate it, you can download the updated theme from here http://www.4shared.com/archive/C_KrFRy8/metrolabtar.html

Edit: copy usr/local/www/themes/twitter_bootstrap_fs /usr/local/www/themes/
Change the theme on the WebGUI
copy the rest to the appropriate location

Pls, follow the above order to not break your WebGUI

Clear-Pixel

@kelsen:

Good afternoon, I've made some changes on metrolab theme, now it's showing alerts in bell icon as shown in screenshot below.

Looks Fantastic ….. glad to see your project is headed in the right direction.

stephenw10

Yep, looks nice. :)

Steve

Clear-Pixel

@ kelsen…..

Here's a excellent idea to hide all the unnecessary clutter top of the status page.

You can hide all the buttons in the Status: Dashboard...... have a down arrow on the right side of the dashboardwhich when clicked will expand theexpose the following.

Widgets button
Welcome dashboard
Add column button
Delete Column Button
Theme Color button

Once theis expanded it will expose the items above, clicking on the arrow in theagain will collapse the unhidden.

As it is now, they are all taking up a lot of unnecessary screen real estate at top of page.

If you don't have a chance to do it, I will take care of the coding soon.

c0urier

@charlie0440 - It's not possible to delete whitelists in the snort package, screenshot: http://sdumps.net/OE

Furthermore when you click Dashboard after you've been in the snort package it links wrongly. Don't know if it's a bug from previously. If you restart Snort you'll also receive some errors.

Have you tested with your theme with the snort package?

kelsen

@Clear-Pixel:

@ kelsen…..

You can hide all the buttons in the Status: Dashboard...... have a down arrow on the right side of the dashboardwhich when clicked will expand theexpose the following.

I see, I tried to do so, but, I just can't find a way to close the, the loader.js is ignoring the.

charlie0440

@kelsen how you trying to do it, jQuery?

kelsen

Ok, I figured out. @Clear-Pixel this is what you wanted?

@charlie0440:

@kelsen how you trying to do it, jQuery?

I just made some changes to the loader.js to insert it after afrom index.php.

Clear-Pixel

Not exactly what I had in mind …..

Thearea which is shaded gray with the text "Status: Dashboard" - The Gray Area - this is where you would nest a hiddenwithin the "Status: Dashboard"

Place a down arrow on right side of this grayarea ...... When clicked it expands the "Status: Dashboard"to expose the buttons.

This can be achieved using CSS3 or jquery .... If you use CSS3 some older browsers could have issues ..... if you use jquery you eliminate compatibility issues.

kelsen

sorry for the misunderstanding, but I think this is not possible since this div appears on all pages, hence the icons appear in all of them without the need, unless I change the code structure.

Clear-Pixel

Hide theunderneath/against the bread crumbor look to see if you can inject the code just for that bread crumb…

Make it look as though it's not there, when its is ...... approach the problem as though it was a magic trick.

If you make the widget draggable that would work for the time being ...... that way you can drag it to the bottom of the page out of the way.

kelsen

ok, how about now?

Edit: I've had to delete the line "Csrf…." from the index.php form, is this a problem?

Clear-Pixel

That's it…. Perfect ...... much cleaner look don't you think?

Don't know what the Csrf line consist of... it could be related to Cross-Site Request Forgeries prevention.

kelsen

yeah, it looks better.

Don't know what the Csrf line consist of… it could be related to Cross-Site Request Forgeries prevention.

looks important ;D
I just know that on 2.0.3 doesn't have this line.

charlie0440

CSRF is a security measure, what it does for example is if you were logged into the GUI via a Chrome browser, someone couldn’t issue a Curl post command, they would have to login via Curl first to get a CSRF.

@kelsen I have some concerns about your work. Let me start by saying it looks amazing and I don’t want to put you off. I do not work for pfsense and have therefore am not speaking for them, the following is my personal view, which may be incorrect!

How do you plan on releasing your theme when finished?

I see two options, a package or to push it to the pfsense team as the only theme to be used (as it breaks the others).

if you add it as a package every new update to pfsense would require fixes on your end, you would also need to be up-to-date with pfsense github at the time of release (which you currently are not, more on this in point 2).
If you want to get it merged into pfsense as a replacement to all the other themes (as currently it breaks all the other themes) I see two issues.

Firstly you need to learn how to use github properly. I realise this is a learning curve and I am willing to help you here, its not that hard.

Your branch is a mess, you have correctly cloned pfsense, but your "Metrolab" branch is based on the 2.1 branch while it should be based on the latest work ie master. This to me is a mistake because your branch is already "439 commits ahead and 582 commits behind master" while admittedly most of these will not effect the GUI some will have been changes which your theme will now be missing, or could be changes which have broken things.

This would also save you work. I see you have merged my progressbar code (https://github.com/kelsen/pfsense/commit/74ff944ce38928be9d43df8d5ebf56bec953d4ff) If you based your branch on master there is no need to merge this separately as it would already be there.

Secondly, if this is a single theme and not compatible with the other themes, all code allowing themeing/skinning etc should be removed. Basically instead of hacking the current working it should be rewritten from scatch. One example would be the loader.js each theme has a different one to allow different code, while that could be hardcoded into the core ie do away will all the themeing stuff.

Sorry for the long post. Its better I say this now then when you finish.

kelsen

CSRF is a security measure, what it does for example is if you were logged into the GUI via a Chrome browser, someone couldn’t issue a Curl post command, they would have to login via Curl first to get a CSRF.

I discovered later that I was copying from google chrome code view, it was a mistake.

How do you plan on releasing your theme when finished?

indeed it must be a package, I'm trying to get help from @marcelloc on this matter; as you said, I've based my work on 2.1, I didn't know what I was doing on github to tell the truth.

My work is intended to 2.1, I realised that would be to difficult to push it to master branch,maybe I can start from scratch for 2.2, I don't know, I'm tired.

Secondly, if this is a single theme and not compatible with the other themes, all code allowing themeing/skinning etc should be removed.

right, I'll take a look.

Basically instead of hacking the current working it should be rewritten from scatch.

This may or not consume all my time, I'll see what can be done, as I said, I'm tired, this is a great effort for me, I admire the pfSense team for their work..

BOUNTY: Web GUI reprogramming

You can hide all the buttons in the Status: Dashboard...... have a down arrow on the right side of the dashboardwhich when clicked will expand theexpose the following.

Widgets button Welcome dashboard Add column button Delete Column Button Theme Color button

Widgets button
Welcome dashboard
Add column button
Delete Column Button
Theme Color button