Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Added modal component #723

Merged
merged 1 commit into from
Apr 23, 2020
Merged

Added modal component #723

merged 1 commit into from
Apr 23, 2020

Conversation

LukasHirt
Copy link
Collaborator

@LukasHirt LukasHirt commented Apr 21, 2020

Screenshot

image

Open tasks:

  • Use variables for colours and gutters
  • Unit tests

@LukasHirt LukasHirt added enhancement New feature or request 2 - Developing labels Apr 21, 2020
@LukasHirt LukasHirt self-assigned this Apr 21, 2020
@@ -17,7 +17,7 @@
</template>

<script>
const req = require.context("@/assets/icons/", true, /^\.\/.*\.svg$/)
const req = require.context("../assets/icons/", true, /^\.\/.*\.svg$/)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Seems that the require-context-hook plugin is not resolving aliases.

@LukasHirt LukasHirt force-pushed the component/modal branch 3 times, most recently from 88122c6 to da74e3b Compare April 22, 2020 21:52
@LukasHirt LukasHirt marked this pull request as ready for review April 22, 2020 23:00
@LukasHirt LukasHirt requested review from PVince81 and kulmann April 23, 2020 07:36
@LukasHirt
Copy link
Collaborator Author

For now, I put into examples also modal with input to show that we'll be able to use it in that way as well. However, as soon as we get rid of this use case in ocis web app, I'd drop this example not to encourage the devs to follow that path.

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 625e511 into master Apr 23, 2020
@delete-merged-branch delete-merged-branch bot deleted the component/modal branch April 23, 2020 09:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3 - To Review enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants