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

add compactType=null to remove vertical packing #21

Merged
merged 16 commits into from
May 25, 2024

Conversation

Gazook89
Copy link
Collaborator

@Gazook89 Gazook89 commented Mar 3, 2024

PR #19 should be merged before this is (this PR includes the changes in #19).

This PR aims at a couple of things:

  1. Add component-specific css for each droppable component. It moves each component into it's own new folder and adds a css file (so for example /components/labelInput/labelInput.css along with /components/labelInput/labelInput.jsx). The CSS within is applied to that component in both the picker and the actual page.
  2. It also adds some styling to the components. This isn't like 'final' styling, it's just what i picked after spending two seconds thinking about it...I just wanted background-color on the components so i could spot them better, for now.

Minor thing, it also changes the hover border to an outline rather than border. This prevents minor layout shifts.

I still need to get input labels to persist, but i'm not sure that will be in this PR or another (other values also do not currently persist).

.empty-space {
height: 100%;
width: 100%;
background: rgb(198, 198, 212);
Copy link
Member

Choose a reason for hiding this comment

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

this should not be here for printing, its an empty space

@5e-Cleric 5e-Cleric added the 🔍 R3 - Reviewed - Awaiting Fixes PR is okayed but needs fixes before merging label Mar 6, 2024
@5e-Cleric 5e-Cleric added this to the Axe 1.0 milestone Mar 6, 2024
@5e-Cleric 5e-Cleric merged commit adcdd81 into naturalcrit:main May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔍 R3 - Reviewed - Awaiting Fixes PR is okayed but needs fixes before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants