Tell us about yourself!
First of all you need to know several things about creating a speaker box. You can't just judge by the size of the speaker unless of course you just want noise. There are acoustical values that need to be taken into consideration. Also the components you hide inside the box all reflect sound waves and alter their paths as well. It's not just as simple as placing everything in a box. You should probably seal the inside of the box and definitely use some padding. You may even need a port to enrich the bass and round out the sound depending on your build. Overall I think you did a nice job but for really good sound you need to know a bit about speaker building.