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/dot 354 design and implement offline component #826

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

Conversation

hanlun0804
Copy link
Contributor

Add an offline component for monoweb's homepage.

Closes: #...

Checklist

  • I have written tests
  • I have provided documentation

Changelog

  • Implementet component from scratch
  • Use position on the maps to make it scroll smoothly

How to test

Place component a convenient place in monoweb and use an array of links to images as parameters. Pass this as both "images" and "pdfs". Component will use an array of links to the frontpage as images and an array of links to the pdfs in the future, when these are possible to access.

Copy link

linear bot commented Mar 7, 2024

Copy link
Member

@henrikhorluck henrikhorluck left a comment

Choose a reason for hiding this comment

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

I think that as a reviewer I should not have to write any code, could you please either add the component somewhere it is used, or make it available to see somewhere else?

For example: this could be moved to the ui-package so that it is shown in ladle (localhost:61000), and it should ideally have some data-fixtures available to check it working

@junlarsen probably has better control of where to place this

Also: this branch should be updated with latest main, ideally with a rebase

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.

3 participants