GoodDollar / GoodCollective

Monorepo for GoodCollective (Segmented UBI and Direct Payments Pool)
MIT License
3 stars 1 forks source link

[Design QA] View Collective / Desktop - Excessive Padding #38

Closed SanaJamm closed 7 months ago

SanaJamm commented 9 months ago

Horizontal gap between stewards and recent transactions should be set to 32 px

Devs:

Screen Shot 2023-11-16 at 10 43 48

Design:

Screen Shot 2023-11-16 at 10 45 38
benefacto commented 9 months ago

Technical Implementation

To set the horizontal gap between the "Stewards" and "Recent Transactions" sections to 32 pixels, you will need to modify the styling of the container that holds these two components. This can be achieved by adjusting the flexDirection and gap properties in your styles.

From the provided code, it seems like the collectiveDesktopActions style is responsible for the layout of these two sections. Here's how you can modify it:

  1. Update the collectiveDesktopActions Style: Adjust the flexDirection to row (if not already set) and set the gap to 32 pixels. This will ensure that there is a 32-pixel gap between the two horizontally aligned components.

    collectiveDesktopActions: {
     flex: 1,
     flexDirection: 'row', // Ensure the components are aligned in a row
     justifyContent: 'space-between', // Adjust as needed for your layout
     gap: 32, // Set the gap between elements to 32 pixels
     // Other styles...
    },
  2. Adjust Child Component Widths: If the child components (StewardList and TransactionList) have fixed widths or maximum widths set, ensure that they accommodate the 32-pixel gap within the total container width. You might need to adjust their widths accordingly.

  3. Test on Different Screen Sizes: After making these changes, test your component on various screen sizes to ensure that the layout appears correctly and the gap is consistent across different devices.

  4. Check for Conflicting Styles: Ensure that no other styles are conflicting with or overriding this layout adjustment. This might include checking parent or child component styles.

  5. Consider Responsive Design: If the layout needs to change for different screen sizes (like mobile vs desktop), consider using media queries or platform-specific styling to adjust the layout accordingly.

By following these steps, you should be able to set the horizontal gap between the "Stewards" and "Recent Transactions" sections to 32 pixels in your ViewCollective component. Remember to test the layout after making these changes to ensure the desired alignment is achieved across different devices and screen orientations.

Estimate

Estimating the time for adjusting the horizontal gap between two sections in a web application involves front-end development tasks, focusing on CSS and potentially responsive design. The task is to modify the styling of a container holding two components ("Stewards" and "Recent Transactions") to set a specific gap.

  1. Understanding the Current Layout (1-2 hours): Examining the existing layout and styling of the collectiveDesktopActions container and the child components (StewardList and TransactionList) to understand how they are currently styled and structured.

  2. Updating the Container Style (1-2 hours):

    • Adjusting the flexDirection and gap properties in the collectiveDesktopActions style to achieve the desired 32-pixel gap.
    • Ensuring that the flexDirection is set to row and the gap is correctly implemented.
  3. Adjusting Child Component Widths (1-2 hours):

    • Modifying the widths of the child components if they have fixed or maximum widths set, ensuring they accommodate the 32-pixel gap within the overall container width.
  4. Testing Across Different Screen Sizes (2-3 hours):

    • Testing the adjusted layout on various screen sizes to ensure that the gap appears consistently and the layout behaves as expected.
    • Checking for any unexpected behavior or layout issues due to the new gap setting.
  5. Handling Responsive Design (1-2 hours):

    • If the layout needs to adapt to different screen sizes, implementing responsive design considerations using media queries or platform-specific styling.
    • Testing these responsive adaptations to ensure they work correctly.
  6. Resolving Conflicting Styles (1-2 hours):

    • Checking for and resolving any styles that might be conflicting with or overriding the new layout adjustments.
    • Ensuring that changes are consistent and do not affect other components or sections adversely.
  7. Review and Refinement (1-2 hours):

    • Addressing any feedback, and making necessary refinements.

Adding these up, the total estimated time could be around 8-15 hours. This estimate assumes that the developer is proficient in CSS and responsive design and that there are no significant unforeseen complexities in adjusting the layout.

vldkhh commented 8 months ago

@krisbitney still reproduces, please re-check. My resolution is 1920x1080

image.png
krisbitney commented 8 months ago

@krisbitney still reproduces, please re-check. My resolution is 1920x1080 image.png

Apologies for this. It should be fixed by this PR: https://github.com/GoodDollar/GoodCollective/pull/62

krisbitney commented 8 months ago

@vldkhh Please QA this when you have a few minutes

L03TJ3 commented 7 months ago

@vldkhh is an older one but still has to be qa'd

vldkhh commented 7 months ago

fixed

krisbitney commented 7 months ago

Closing as completed, based on @vldkhh 's comment