Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CHANGE][NMP-625] Update Crop Details form layout for sawdust question. #639

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

fergmac
Copy link
Contributor

@fergmac fergmac commented Mar 22, 2024

…ss browsers breaking tooltip layout.

Pull Request Standards

  • The title of the PR is accurate
  • The title includes the type of change [HOTFIX, FEATURE, etc]
  • The PR title includes the ticket number in format of [NMP-###]
  • Documentation is updated to reflect change [README, functions, team documents]

Description

This PR includes the following proposed change(s):

  • Bump sawdust question onto it's own row. So that we don't need to position the tooltip relatively. The relative position is what is causing the issue due to no CSS reset and Bootstraps' Normalize not normalizing some elements of HTML forms across browsers.

Screenshot 2024-03-22 at 8 40 50 AM

Screenshot 2024-03-22 at 8 43 00 AM

@fergmac fergmac requested a review from acatchpole March 22, 2024 20:56
@fergmac fergmac self-assigned this Mar 22, 2024
acatchpole
acatchpole previously approved these changes Mar 26, 2024
@fergmac
Copy link
Contributor Author

fergmac commented Apr 2, 2024

firefox:
Screenshot 2024-04-02 at 10 19 00 AM

safari:
Screenshot 2024-04-02 at 10 19 48 AM

@fergmac fergmac merged commit 749f1ed into master Apr 2, 2024
1 check passed
@fergmac fergmac deleted the nmp-625-tooltip-adjustments branch April 2, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants