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

style: Add small separation in scan barcode button #99

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

duhow
Copy link

@duhow duhow commented Jan 4, 2024

Adds a slight separation between buttons. Can be increased to mt-2 but this should work for mobile devices.

image
image

@duhow duhow changed the title Add small separation in scan barcode button style: Add small separation in scan barcode button Jan 8, 2024
@@ -25,7 +25,7 @@
</v-item-group>
</h3>
<v-sheet v-if="productMode === 'barcode'">
<v-btn class="mb-2" size="small" prepend-icon="mdi-barcode-scan" @click="showBarcodeScanner">Scan a barcode</v-btn>
<v-btn class="mb-2 mt-1" size="small" prepend-icon="mdi-barcode-scan" @click="showBarcodeScanner">Scan a barcode</v-btn>
Copy link
Member

@raphodn raphodn Jan 12, 2024

Choose a reason for hiding this comment

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

mt = margin-top (docs)

did you mean mr ?

Copy link

Choose a reason for hiding this comment

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

has this been fixed?

Copy link

Choose a reason for hiding this comment

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

@raphodn I think @duhow wants to add margin top to make space between "Barcode" / "Category" buttons and "Scan a Barcode button"

@Valimp
Copy link

Valimp commented Feb 20, 2024

@duhow Are you still working on it ?

@duhow
Copy link
Author

duhow commented Feb 20, 2024

Nope, feel free to close or push any needed changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants