Invoice App | React, Next.js, styled-components, Formik, Framer Motion

Hey APG,

Congrats on completing your 30th (!!) challenge overall and your 1st Guru level challenge! I'd agree with Em and say this is your best one yet (although you've posted lots of brilliant solutions!) 🎉

Here are some thoughts related to some of the bugs and issues you mentioned in your README.md:

• You can use lots of different approaches to structure React applications, as I'm sure you've noticed! On Frontend Mentor, I use an altered version of a domain-driven architecture. This is mostly because we use Redux for state management. If I were starting fresh today, I'd probably go a different route. The way you've structured your code works well for this size project, though. It's easy to navigate, you've separated the components nicely, and you've created good utility functions.
• I'd agree that using CSS custom properties instead of using Styled Component themes is a better approach. I'm feeling this pain on the Frontend Mentor codebase and will look to refactor at some point!
• The quick flash of light will most likely come from the fact that the app needs to pull the state from localStorage. So it's partially rendered by the time it can read the state. In a typical app where you're pulling data, this configuration would be stored as a user preference in the database. This means it wouldn't have the flash, as everything would be set during server-side rendering in Next.js. That might not be the issue, but that would be my guess anyway.
• You're right about the custom dropdown not being accessible. As it is, the focus never goes into the dropdown if navigating by keyboard. This is because you've got display: none; on the input elements. Instead, I'd recommend using a .sr-only class to visually hide the checkboxes and keep them accessible to screen readers. Here's a good article from the A11y Project about visually hiding content to learn more.

After looking at your code, here are a few recommendations:

• Now that optional chaining in JS is a thing, you don't need to use the && operator so much. That means this line: filteredInvoices && filteredInvoices.length === 0 could become this filteredInvoices?.length === 0.
• You're using a div with span elements to display the invoice page's addresses. Instead, you could add more context in the HTML by using the address element. You can read more about the address element on MDN here.
• You could add the media query breakpoints to your theme. Seeing as they're used throughout the app, this would reduce duplication of values.

It's awesome to see you adding animations and extra details like the attribution overlay. Your attention to detail is incredible!

A couple of feature suggestions I'd love to see would be:

• Try making the form disappear if someone clicks away from it instead of always needing to click the buttons at the bottom.
• I know this wasn't in the design, but you could remove the "Mark as Paid" button from invoices that have a "Paid" status.

Overall, you've done an AMAZING job on this challenge! Especially considering you're still new to React and this was your first time using most of these packages.

Keep up the brilliant work! 🙌

This is an excellent job. Keep coding.

Wow! Hello ApplePieGiraffe!

Amazing work! Hopefully, I'll be to create complex projects like this one too! I'll bookmark it for future reference too :) Thank you!

Hey, APG!

I'm just reading here throw all the above feedback, and it's so warm and inspiring, also like overall community here. It's very valuable experience to have a touch with such folks like you, thanks for that :)

I'm pretty new here, try out your app something, and have a small question. Is it suggested any validation on form fields content? I just tapped my keyboard randomly a lot and just three fields produce some warnings, actually number data fields (so, I typed into those fields very big numbers, my invoice had a price over 10e+40 though...), and also e-mail field. Also my colleague told me a story, when he while testing the web app sent around 2Gb strings in authorization forms :D So, is it suggested such kinds of validation mechanisms to apply into Frontend Mentor challenges, or is it an overhead for this type of tasks and not suggested here?

Happy coding :)

Edit: here are some pictures with artifacts of such an irresponsible input :D https://drive.google.com/drive/folders/1DWqMXkqhrISK1GLCX2N2tE8rEul9wNcC?usp=sharing

This is excellent APG! I think it's your best one yet. Great attention to detail. Smooth transitions and I like the little extras, like the number of invoices on the browser tab and the surprise when clicking on your avatar 😀 Hopefully someone comments regarding architecture since I'm curious about that as well. Filter works perfectly for a single category, my feature request would be to add for multiple categories lol :-)

hello APG!

I love the cool animations and you also did an excellent job!

Keep Coding!

                          (@@)
                             \__/

Wow!! This is so awesome.

You did a fantastic job from the animations to the light and dark themes to the filters and responsiveness. I will definately take time to learn how all this was achieved from your repo. And congrats for submiting your guru solution.

I'm stuck on your projects. Amazing stuff. I love the way you document them on github. Gave me some ideas. I've been curious about the dark theme switch thingy. Will take time to study how you achieved the functionality. Maybe you could apply margins to the filter options to make it more breathable and easily distinguishable

Hey ApplePieGiraffe!

I'm working on the same project and I will submit it in a few days. Your build is awesome, it's very responsive and reactive. However, using react dev tools, I saw that a lot of re-renderings happening on your application, mostly from components on your Invoice page. So I started looking at your code and I saw that you're not using memo or useCallback on your functions and components. What's happing is that every time the state of your parent element changes, all the components are going to re-render again, even if they don't need to re-render. If possible, I would like to copy your repo and create a solution for this problem. Let me know if I can, and maybe, we could work together to solve this problem!

Thanks for your time and happy coding :)

Well done ApplePieGiraffe! I just got out of the hospital, so I don't have time to dive in to your code, but I've bookmarked it for future reference.

This is soo neat! APG, I'll use your site as the inspiration for the animations. This is phenomenal work!!! <3

FWIW, I'm currently working on this project and have made it into a full stack app with React/Nextjs and Fauna (including authentication and with plans to turn it into a SaaS). I'm looking forward to deploy it in a few weeks.

Hey, APG! Congrats on your 30th solution! 🥳🎉

I'm nowhere near the point where I could give this a proper critique, and it seems like you've already gotten some amazing feedback, so I guess I'll just pile on the praise bandwagon! 😄

The animations and transitions are fantastic as always - the UX feels great. I know you've utilized various versions of the attribution button/avatar in previous projects, and it remains a great touch (and something I might have to steal!).

Definitely bookmarked for reference. 👍 Looking forward to the next one!

Hey ApplePieGiraffe 🙂 !

Congratulations, the project is just on top ! Respect👏.

Hi, ApplePieGiraffe👋

Congratulations, another amazing solution from you (as expected). I love the soft and cool animations you used. Your work impresses me, again, congratulations, mate ;).

Keep coding🧡

Hello APG

This is a great version of the Invoice App, i couldn't find big issues playing with the invoices and testing the responsive layouts.

Only thing i noticed is that the new invoice animation vs edit invoice animation are at different speeds. To open and close an existing invoice takes a bit long... but the animations are sweet! :)

Looking forward to solve this challenge myself someday.

Honestly I really admire your work, especially what you do with animations, I'm getting there someday and you're juss motivating me to get there. I love the dark and light theme, well done APG!! Mobile design is pretty solid.