Skip to content

Conversation

@PranathiChillarige
Copy link
Contributor

@PranathiChillarige PranathiChillarige commented Oct 6, 2024

Fixes - #357

Description -
This PR aims to avoid the blocking of Action buttons(preview button, Export button and rows input field)
with underneath content(add another field button) in mobile and desktop view by

1 . creating a scrollable section for fields.
2 . adjusting the margins/bottom in between to update the page layout properly.
3 . modifying initialFields to 3 as it has a scrollable section and also to avoid clumsiness on the page layout.
4 . hidden Field header(Field name, Field type) when there are no fields as these headings are not required to be shown when there are no fields.

Screenshots -
mobileView
desktopView

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

Please feel free to suggest any changes, will update accordingly. Thank you.

@vercel
Copy link

vercel bot commented Oct 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-dev-tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2024 5:39am

@PranathiChillarige PranathiChillarige force-pushed the fix/blocking-of-preview-button-in-json-generator branch from b67d22c to a4ff0e9 Compare October 6, 2024 05:38
@Bashamega Bashamega requested review from annuk123 and removed request for Bashamega October 6, 2024 08:00
@annuk123 annuk123 merged commit f6d6c2d into Bashamega:main Oct 6, 2024
@PranathiChillarige
Copy link
Contributor Author

Hey @Bashamega Can we close this issue now?

@annuk123
Copy link
Collaborator

annuk123 commented Oct 7, 2024

Hey @Bashamega Can we close this issue now?

Nope

@Bashamega
Copy link
Owner

Hey @Bashamega Can we close this issue now?

Nope

I have closed it

@annuk123
Copy link
Collaborator

annuk123 commented Oct 7, 2024

Hey @Bashamega Can we close this issue now?

Nope

I have closed it

okay, Actually there is an issue, that's why I told

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants