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

Feature: Adding a display option in the backend for specific use cases #53

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

SebastianUhlig
Copy link

This PR has both a feature and a small fix.

The fix takes care of the slightly failed display of table headers in the DetailField when there are no entered names.
I also removed the static 2rem heights per row that led to unfortunate cases when you wanted to go to "show content".

image

The feature ->colsPerRow(true or false) adds a function to SimpleRepeatable to change the display to rows in the backend. Very useful if you want to have multiple fields under each other and not squeezed next to each other. This was very useful for us, because you ended up not being able to use the text field at all.

image

@Norgul
Copy link

Norgul commented Jul 18, 2024

@SebastianUhlig I'd love this feature. I tried copying your PR contents in my implementation (using a custom fork of my own, so can't just do a checkout), however it is looking weird (using Nova 4). Did you do any improvements since this PR was made?

image

@Norgul
Copy link

Norgul commented Jul 18, 2024

Nvm, I figured it out. Classes beginning with nsr must be changed to o1

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