Hey, just wanted to share the fake bank website I made as a personal project(not for FCC). It’s very basic and very silly but I had a good time with it. It is just HTML/CSS and very basic JS. You can “log in” to the account with the username “fake” and the password “bank” and it takes you to an accounts page where you can transfer money between accounts. The JS updates the account balances and adds a recent transfer line at the bottom. You can also select the different accounts from the dropdown menu and it shows a randomly generated list of recent transactions. Check it out, I would appreciate any feedback or suggestions for what else to add. Be nice, I’m still very new at this.
For the sign in feature, I think you should add some logic to your code that prevents users from clicking the button without putting in any information.
Right now, it takes me to the account balance sheet.
I think the buttons should have a cursor pointer.
For your javascript, you might consider looking into ways of refactoring your script.js.
Right now, you have a nested for loop with a lengthy if else statement which can be hard to follow.
Welcome to the forums @ksmith52. Nice job on your page, looks like you had fun with it.
Some things you may want to revisit;
User’s should be able to click the label to toggle the selection, not just the radio button.
Seeing as how you’re using HTML5 is there a reason you used table rather than taking advantage of CSS flexbox or grid as needed?
Speaking of, the tc element was used by Microsoft Office a while ago but I don’t recall it ever being an element in HTML
You can run your CSS code through the W3C validator.
Honestly, the menu got a bit muddled as I was trying to get everything to line up the way I wanted it to. I tried making it a table while I was playing around with it and it worked well enough so I left it. I’ll work on a better alternative.
Now that you mention it, I’m not sure where I got the idea to use “tc” from. Maybe that was some old office knowledge coming through, haha.
