Show DevBest thoughts/suggestions on my personal site

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
please move this thread if it's in the wrong section :up:

Been working on my personal cms for awhile and i managed to finish some pretty unique features.
z7jpAsG.png

Don't take that warning message so lightly, it actualy means what it says.

Cross-site request forgery is often forgotten when developers write their code, so i made sure i'm protected against it.
V82aJz4.png

there will be a hidden token generated every instance of the page load. if a request is made and this token is changed at all, it will stop the request and log the ip etc into the database.
8pjjXLp.png


Since this is an admin panel, i'm going to log failed login attempts aswell.
xYRHRJ2.png


The ability to ban certain ip addresses is also done
2sSYszI.png

qraZ8og.png


maintenance is also finished
jHgxSM4.png

oADBgQO.png


I'm also using prepared statements to ensure i'm protected from sql injections.
PHP:
query = Database::dbConnect()->prepare("SELECT username, password FROM gweb_users WHERE username=:username AND password=:password");
        $query->bindParam(':username', $secure_user);
        $query->bindParam(':password', $secure_pass);
        $query->execute();
  
        if($row = $query->fetch()){
      
            $_SESSION['username'] = $row['username'];
            header("Location: admin_dash");
        }

Thoughts? how can i make the design prettier (i suck at designing) and what other functions should i include? cheers
 
Last edited:

Benden

maging ang maganda mamatay
Jun 4, 2010
2,286
1,482
Please capitalize the letters in the database rows, it looks gross without it. Also, I'm not digging the font you went with. In terms of design it would like nicer if you added icons to the navigation and a small logo beside AdminPanel
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
Please capitalize the letters in the database rows, it looks gross without it. Also, I'm not digging the font you went with. In terms of design it would like nicer if you added icons to the navigation and a small logo beside AdminPanel
Thanks for the suggestions, i'll definetly go with some icons for sure. Know of any good fonts?
 

Sysode

Front-End Developer
Dec 11, 2012
1,673
848
Thread moved to Showcase, purely for the fact you're asking for feedback and this is personal.

Should you release this (or plan to) please update the thread and I'll move it to the necessary destination.
 

BIOS

ಠ‿ಠ
Apr 25, 2012
906
247
The design doesn't look too bad providing you've just started the development of it, however some things could look better such as the input boxes as they are still using the default design.

Not too sure on the back-end of the project as there's not much to look at, but from what I can see it seems you are performing redirection without killing the script. Don't forget to use either exit(); or die(); after performing the redirection to ensure that the code following the redirection isn't executed.
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
Thread moved to Showcase, purely for the fact you're asking for feedback and this is personal.

Should you release this (or plan to) please update the thread and I'll move it to the necessary destination.
Thanks man
The design doesn't look too bad providing you've just started the development of it, however some things could look better such as the input boxes as they are still using the default design.

Not too sure on the back-end of the project as there's not much to look at, but from what I can see it seems you are performing redirection without killing the script. Don't forget to use either exit(); or die(); after performing the redirection to ensure that the code following the redirection isn't executed.
gotcha

had some help from the community yesterday, messed around with the style more:
MxDpKU3.png

global font changed to tahoma, and the header font is some hippie one from google font hosting. sidebar icons are now present, and i raised the header box titles to 10px padding (thanks @Canadian )
also added some animations to the sidebar:
ezgif-689684459.gif
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
I think it looks neat. Well, yeah kinda does.

What is it for?
Thanks, and it's for my personal site. i actualy still have a blank index, only thing i've been working on is this admin panel lol, cheers
 

BIOS

ಠ‿ಠ
Apr 25, 2012
906
247
Thanks man

gotcha

had some help from the community yesterday, messed around with the style more:
MxDpKU3.png

global font changed to tahoma, and the header font is some hippie one from google font hosting. sidebar icons are now present, and i raised the header box titles to 10px padding (thanks @Canadian )
also added some animations to the sidebar:
ezgif-689684459.gif
Improving, looks a lot better with the edits especially the icons.

Try to style the form inputs on the above image which are used for the "Ip Address" & "Reason for ban", maybe round the borders of them and clean them up?
 

BIOS

ಠ‿ಠ
Apr 25, 2012
906
247
thanks, i'll give this a go and see how it turns out
 
D8FuekJ.png

I'm digging it
 
Live demo:
user: devbest
pass: demo

let me know what you think
Just had a quick look and found one thing, make sure you check if it exists first as it may throw errors if there are no records:
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
Just had a quick look and found one thing, make sure you check if it exists first as it may throw errors if there are no records:
true, going to fix that right now.
 
fixed! thanks for letting me know
gE2BbFc.png

 
So I think i'm satisfied with my site now, mods close this thread if it's nesseccary because i'm finished with this.

voG7xAU.jpg


VYhJeGz.png


thanks to everyone who helped me out, cheers
 

brsy

nah mang
May 12, 2011
1,530
272
Looks really nice man; I might do something like this just for the experience. I'm also not too fond of your data tables. The colors and the padding puts me off a bit.

Reduce the padding for the table body, and mess with some colors for the table heading. Maybe a darker shade of your original shade of blue.
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
Looks really nice man; I might do something like this just for the experience. I'm also not too fond of your data tables. The colors and the padding puts me off a bit.

Reduce the padding for the table body, and mess with some colors for the table heading. Maybe a darker shade of your original shade of blue.
cheers man i'll fiddle with it. I'm thinking of making a counter for login attempts, because last night as demonstrated by @Khalil my security logs was getting spammed. perhaps after 2 attempts i could make it so their ip gets auto banned.
 

Khalil

IDK
Dec 6, 2011
1,642
786
cheers man i'll fiddle with it. I'm thinking of making a counter for login attempts, because last night as demonstrated by @Khalil my security logs was getting spammed. perhaps after 2 attempts i could make it so their ip gets auto banned.
Why you even need a control panel for a portofolio is beyond me.
 

griimnak

You're a slave to the money then you die
Jul 20, 2013
957
800
Why you even need a control panel for a portofolio is beyond me.
it's for the learning experience, i'm trying to extend my knowledge in php :rasta:
ay atleast i'm not burning braincells on habbo
 

Users who are viewing this thread

Top