Jamie Big
@jamesekunolaAll comments
- @S0PHI3KSubmitted over 1 year ago@jamesekunolaPosted over 1 year ago
Your UI is impressive, yet a few adjustments are necessary:
- Avoid using a fixed height for the
card-container
andthank-you-container
divs to address the overflow problem. - Prevent users from submitting a rating if they haven't selected a rating number.
- Implement a background color of
var(--orange)
to indicate the selected rating number.
These modifications will enhance the user experience and the overall functionality of your UI.
0 - Avoid using a fixed height for the
- @semperprimumSubmitted over 1 year ago
I would greatly appreciate any feedback you may have!
- I utilized Formik for form validation. This particular aspect is something I'm uncertain about since it was my first time working with it. Initially, I attempted to create my own form validation by implementing hooks, and I achieved some success. However, I eventually decided to switch to Formik as it became increasingly frustrating.
- I'm quite unsure about the positioning of both cards. I employed
position: absolute
for them, but I wonder if usingtransform
properties might have been a better choice. Additionally, I used absolute positioning for everything inside the card. Did I perhaps overuse it?
@jamesekunolaPosted over 1 year agoHi Bogdan, for some reason your page is not displaying. it gives an error message of "Deceptive site ahead" i think you should change the domin name
0 - @abolore1Submitted over 1 year ago
How to achieve DRY but later figured it out. I wonder if there's always a link to a well finished live url which will let us know we have actually come up with all the task entails.
@jamesekunolaPosted over 1 year agoHi Otutu,
It appears that there are some improvements needed in your UI and functionality. Here are the points you should work on:
Regarding the UI for tablet and desktop screen sizes, it seems like you haven't coded it yet. You still need to work on the responsive design for these screen sizes.
For functionality, let's focus on step one:
-
Validation: Ensure that users can't proceed to step two if certain conditions are not met. For example, if the name input contains numbers, if the email input lacks "@" or valid domain endings like ".com" or ".co", or if the phone number input contains letters or is not a valid number. You can address these issues using regex (regular expressions).
-
Retain entered values: When users click on the "Go Back" button, the values they previously entered should still be visible, allowing them to make changes if needed.
Moving on to step two:
-
Default selection: The Arcade plan should be pre-selected by default when users arrive at this step.
-
Plan selection: Users should have the ability to choose and change one plan out of the three available options. You can achieve this by using radio buttons.
-
Retain selected plans: Similar to step one, when users click the "Go Back" button, the plans they selected in step two should still be visible.
Lastly, in step three:
-
Default add-ons: Two of the add-on plans should be checked by default.
-
User interaction: Users should be able to check and uncheck the add-ons by clicking on the plans themselves, not just on the checkboxes.
I hope these suggestions help you improve your UI and functionality. Good luck with your project! If you have any further questions, feel free to ask.
Marked as helpful0 -
- @elian-devSubmitted over 1 year ago@jamesekunolaPosted over 1 year ago
Hi Geordy, your ui look ok, but there are some issues you might need to address.
1 There is horizonatal overflow when device screen size is around 830px 2 users shouldn't be able to go to the next step if phone number isn't entered 3. The next button disapper when input is focused in page 1 on opera mini browser 3 input value for name shouldn't contain numbers or special characters. 4 in the add-ons on page 3 user should be able to still see their selected choice when they come back to page 3. 5 The color of the input when typing should be grey.
overall your ui is great, i love how you added animination to the pages. great work
Marked as helpful0 - @jamesekunolaSubmitted over 1 year ago
built using redux
@jamesekunolaPosted over 1 year agoAdded a country select for the phone input, all that is left out is refactoring the code. Thanks y'all for feedback, I had to test the app on all browsers to see how it look. I found out that the checkbox box display different on opera mini browser.
0 - @jamesekunolaSubmitted over 1 year ago
I thoroughly enjoyed working on this project, although I did encounter some challenges on the technology page. One specific hurdle was determining how to appropriately display portrait or landscape images based on the user's screen size. I would greatly appreciate any feedback on what additions you believe should be made to the page.
@jamesekunolaPosted over 1 year ago"I've incorporated animations for the explore button and the rest of the page. However, I'm encountering a problem on mobile devices where there is overflow on both the horizontal and vertical axes, even though I've applied
{overflow-x: hidden}
to the body. I would greatly appreciate any advice or suggestions on how to resolve these issues."0 - @MayckL2Submitted over 1 year ago
Responsive landing page with display flex and media query, switching flex-direction and width depending on image size.
@jamesekunolaPosted over 1 year agoThere is a horizontal overflow issue in your UI, and it seems that you might have forgotten to set a maximum width for your UI. Please address these concerns, particularly when the user's screen size exceeds 90rem. Apart from these points, your user interface is fantastic.
0 - @elvissamuelSubmitted over 1 year ago
I'm currently learning ReactJS so I decided to build this with React. I simply learned how to reuse components. One funny challenge I always have when developing a website is picking the right font and its color, I'd appreciate any advice on how to improve in it.
@jamesekunolaPosted over 1 year agoTo prevent your UI from extending to the full width of the page when the screen width exceeds 90rem, it is advisable to add a maximum width constraint. This will ensure that your UI maintains a reasonable width even on larger screens and you should add a favicon icon instead of the default react favicon.ico. Other than that, your user interface is commendable.
0 - @elvissamuelSubmitted over 1 year ago
Bulk of the work was in the Hero Section and it was good for practice
@jamesekunolaPosted over 1 year agoThere is a horizontal overflow issue in your UI, and it seems that you might have forgotten to set a maximum width for your UI. Please address these concerns, particularly when the user's screen size exceeds 90rem. Apart from these points, your user interface is fantastic.
Marked as helpful1 - @elvissamuelSubmitted over 1 year ago
It was a pretty interesting project, I wanted something that will require me calling an endpoint, and I learnt a lot in building this project
@jamesekunolaPosted over 1 year agoYou haven't included a transition effect when the user switches the theme, and the favicon icons are not visible. Please take a look at these issues. Your UI design is impressive.
0 - @ArmanidrisiSubmitted over 1 year ago@jamesekunolaPosted over 1 year ago
Resolve issue where clicking on a country card causes a page break instead of displaying the details page. This issue needs to be addressed.
Marked as helpful0 - @Oneluv1Submitted over 1 year ago@jamesekunolaPosted over 1 year ago
"Fix issues with search feature and region filtering functionality; consider implementing dark mode theme. Overall, the code functions properly."
0