vikunja/vikunja
5
49
Fork 89
Code Issues 132 Pull Requests 33 Packages Releases 37 Activity

Frontend: Task List Visual Improvements #2264

Merged
konrad merged 4 commits from Elscrux/vikunja:feature/improved-task-lists into main 2024-04-11 15:46:45 +00:00
Conversation 3 Commits 4 Files Changed 2 +8 -2
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
frontend/src/components/input/editor/TipTap.vue
View File

@ -891,8 +891,14 @@ ul[data-type='taskList'] {
padding: 0;
margin-left: 0;
li[data-checked='true'] {
color: var(--grey-500);
text-decoration: line-through;
}
li {
display: flex;
margin-top: 0.25rem;
> label {
flex: 0 0 auto;

4
frontend/vite.config.ts
View File

@ -17,11 +17,11 @@ import postcssEasings from 'postcss-easings'
import postcssEasingGradients from 'postcss-easing-gradients'
const pathSrc = fileURLToPath(new URL('./src', import.meta.url))
const pathSrc = fileURLToPath(new URL('./src', import.meta.url)).replaceAll('\\', '/')
dpschen commented 2024-06-26 09:54:27 +00:00
Review
Copy Link

👋 Hey @Elscrux:

out of interest: what was the replacement of '\' with '/' for?

👋 Hey @Elscrux: out of interest: what was the replacement of '\\' with '/' for?
Elscrux commented 2024-06-26 17:06:47 +00:00
Review
Copy Link

This was required to make the build work locally on my machine. Otherwise the files couldn't be read apparently.

This was required to make the build work locally on my machine. Otherwise the files couldn't be read apparently.
dpschen commented 2024-06-26 17:35:24 +00:00
Review
Copy Link

Thanks.
Was there a file path displayed that caused this error?

I'm asking because I remember that the path of some imported component contained //. I fixed that just recently (can't find commit right now, but if necessary I can look it up).

Thanks. Was there a file path displayed that caused this error? I'm asking because I remember that the path of some imported component contained `//`. I fixed that just recently (can't find commit right now, but if necessary I can look it up).
Elscrux commented 2024-06-26 17:37:28 +00:00
Review
Copy Link

I'm afraid I don't remember the error

I'm afraid I don't remember the error
👍 1
// the @use rules have to be the first in the compiled stylesheets
const PREFIXED_SCSS_STYLES = `@use "sass:math";
@import "${pathSrc}/styles/common-imports";`
@import "${pathSrc}/styles/common-imports.scss";`
const isModernBuild = Boolean(process.env.BUILD_MODERN_ONLY)
const legacy = isModernBuild