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

Introducing button box #332

Conversation

spaceo
Copy link
Contributor

@spaceo spaceo commented Nov 4, 2023

Link to issue

https://reload.atlassian.net/browse/DDFLSBP-57

Description

This PR introduces the button box and sticky button box concept.
The footer box classes were created in order to accomodate the sticky renewal button on the loan page when watching it on a mobile device.
The button box classes replaces the modal-loan__buttons classes, since button box is more generic and used not only in modals.

Screenshot of the result

button_loans.mov

That is meant for items that only should be shown on mobile
@spaceo spaceo changed the title Ddflsbp 57 mobilvisning knap til forny flere vises ikke Introducing footer box Nov 5, 2023
@spaceo spaceo marked this pull request as ready for review November 5, 2023 04:58
@spaceo spaceo force-pushed the DDFLSBP-57-mobilvisning-knap-til-forny-flere-vises-ikke branch from 31facfb to 78f35d3 Compare November 5, 2023 10:59
@spaceo spaceo changed the title Introducing footer box Introducing button box Nov 5, 2023
Since we need the expression in different places besides modals we can
generalise it
above the renew button on the loan list page on mobile devcices
@spaceo spaceo force-pushed the DDFLSBP-57-mobilvisning-knap-til-forny-flere-vises-ikke branch from 78f35d3 to 2c61224 Compare November 5, 2023 17:49
Copy link
Contributor

@Adamik10 Adamik10 left a comment

Choose a reason for hiding this comment

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

Very nice :))
Although I still don't think I get what the "button-box" is from just the naming. 🙈

@spaceo spaceo merged commit a887985 into danskernesdigitalebibliotek:develop Nov 9, 2023
6 checks passed
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.

5 participants