Skip to content

Calibration card and PV input styling#1695

Merged
DevonRD merged 3 commits intoPhotonVision:mainfrom
DevonRD:main
Jan 8, 2025
Merged

Calibration card and PV input styling#1695
DevonRD merged 3 commits intoPhotonVision:mainfrom
DevonRD:main

Conversation

@DevonRD
Copy link
Contributor

@DevonRD DevonRD commented Jan 8, 2025

Images are before and after comparison.
Does the following:

  • Fixes several styling issues with pv-* input elements, including top padding, vertical alignment, and allocated input width
    image
    image

  • Conforms the calibration details modal to overall styling and spacing standards
    image
    image
    (left the blank table there on empty calibrations to give the user a sense of what they might see if they don't have any)

@DevonRD DevonRD requested a review from a team as a code owner January 8, 2025 21:02
Copy link
Contributor

@mcm001 mcm001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretttyyyyyy

@DevonRD DevonRD merged commit e40c8fb into PhotonVision:main Jan 8, 2025
mcm001 pushed a commit to mcm001/photonvision that referenced this pull request Feb 18, 2025
Images are before and after comparison.
Does the following:
- Fixes several styling issues with pv-* input elements, including top
padding, vertical alignment, and allocated input width

![image](https://github.com/user-attachments/assets/70d37e65-e0cd-4c71-8ea1-941ec2175850)

![image](https://github.com/user-attachments/assets/031d008d-3cce-4ed2-bc88-5fbecf20c94f)

- Conforms the calibration details modal to overall styling and spacing
standards

![image](https://github.com/user-attachments/assets/18281551-f924-4e12-9ad4-d2ec470dbc70)

![image](https://github.com/user-attachments/assets/db772325-7650-467d-8521-252c7d38601f)
(left the blank table there on empty calibrations to give the user a
sense of what they might see if they don't have any)
@Gold856 Gold856 added frontend Having to do with PhotonClient and its related items styling Issue pertaining to styling issues in the PhotonClient labels Jul 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Having to do with PhotonClient and its related items styling Issue pertaining to styling issues in the PhotonClient

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants