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

Make form submit full width fixed #254

Merged
merged 6 commits into from Jul 20, 2021
Merged

Make form submit full width fixed #254

merged 6 commits into from Jul 20, 2021

Conversation

cdubz
Copy link
Member

@cdubz cdubz commented Jul 14, 2021

Closes #231. Alternative to #249.

@lutzky @BenjaminHae @ntrecina interested in your thoughts on this approach (since y'all chimed in on #249 to some extent). Mobile experience seems solid but I'm not sure how I feel about desktop -- the ux is not better there but I also don't think it is worse.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 99.191% when pulling dca03f4 on fixed-submit-button into 4116419 on master.

@cdubz cdubz temporarily deployed to babybuddy-pr-254 July 14, 2021 04:01 Inactive
@lutzky
Copy link
Contributor

lutzky commented Jul 14, 2021

I think it's slightly worse on desktop; ideally this would only occur on mobile, but I think it's more complicated to get that to work within bootstrap, and not necessarily worth it, so I'm in favor of this change.

@cdubz
Copy link
Member Author

cdubz commented Jul 14, 2021

I don’t think it would be much effort to make this more responsive. This was just the lightest touch approach. I’ll sit on it for a moment.

@ntrecina
Copy link
Contributor

@cdubz I like the ux on mobile but I'm also on the fence about desktop. I don't think it's terrible but it does feel a bit off. I think if you could split the experience between mobile and desktop that would be perfect.

@cdubz cdubz temporarily deployed to babybuddy-pr-254 July 20, 2021 03:26 Inactive
@cdubz cdubz temporarily deployed to babybuddy-pr-254 July 20, 2021 03:44 Inactive
Troubleshooting issue with Pipenv Python executable discovery.
@cdubz cdubz temporarily deployed to babybuddy-pr-254 July 20, 2021 03:51 Inactive
@cdubz cdubz merged commit 371d6f5 into master Jul 20, 2021
@cdubz cdubz deleted the fixed-submit-button branch July 20, 2021 03:55
@cdubz cdubz added this to the v1.7.1 milestone Jul 20, 2021
@cdubz cdubz modified the milestones: v1.7.1, v1.8.0 Aug 5, 2021
@cdubz cdubz added the enhancement Feature requests or improvements to existing functionality label Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests or improvements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't hide Submit button below optional fields
4 participants