vikunja
/
frontend
Archived
6
39
Fork 49
Code Issues 1 Pull Requests 4 Projects Releases 26 Activity

feat: restyle unauthenticated screens #1103

Merged
dpschen merged 29 commits from feature/login-pages into main 2021-12-12 16:40:14 +00:00
Conversation 12 Commits 29 Files Changed 10 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit babfd24739 - Show all commits

2
src/components/misc/no-auth-wrapper.vue
View File

@ -61,7 +61,7 @@ const motd = computed(() => store.state.config.motd)
padding: 1rem;
konrad marked this conversation as resolved Outdated
dpschen commented 2021-11-28 15:43:08 +00:00
Outdated
Review
Copy Link

Use mobile first:

.image {
    @media screen and (max-width: $tablet) {
        display: none;
    }
    @media screen and (min-width: $tablet) {
        width: 40%;
        background: url('@/assets/no-auth-image.jpg') no-repeat bottom/cover;
        position: relative;
    }
    @media screen and (min-width: $desktop) {
        width: 60%;
    }
}

Compress image and load with something like https://github.com/JonasKruckenberg/imagetools

Use mobile first: ```scss .image { @media screen and (max-width: $tablet) { display: none; } @media screen and (min-width: $tablet) { width: 40%; background: url('@/assets/no-auth-image.jpg') no-repeat bottom/cover; position: relative; } @media screen and (min-width: $desktop) { width: 60%; } } ``` Compress image and load with something like https://github.com/JonasKruckenberg/imagetools
konrad commented 2021-12-11 19:44:06 +00:00
Outdated
Review
Copy Link

Compress image and load with something like https://github.com/JonasKruckenberg/imagetools

With the vite wrapper and use it for other images as well?

> Compress image and load with something like https://github.com/JonasKruckenberg/imagetools With the vite wrapper and use it for other images as well?
dpschen commented 2021-12-12 15:02:55 +00:00
Outdated
Review
Copy Link

Yes. But maybe let's move this to a new issue.

Yes. But maybe let's move this to a new issue.
👍 1
konrad commented 2021-12-12 15:03:23 +00:00
Outdated
Review
Copy Link

yeah I think we should.

yeah I think we should.
display: flex;
flex-direction: column;
justify-content: end;
justify-content: flex-end;
&.has-message {
justify-content: space-between;