Bug Report/UX Issue: Merging prefers merged contents colours
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- Nửa ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- rust
- Lĩnh vực
- computer-graphics
Hướng nghiên cứu
The report points at the Merge node in the node graph as the place where fill and stroke are resolved, with the bottom input's values overriding the top input's. Reproduce it by setting a fill or stroke colour on the top input and checking the merged result, then find where the Merge node chooses between its inputs. Done means agreement with maintainers on whether the top or bottom input should win, and a fix that matches that decision. The thread has not settled whether this is a bug or a UX question, so confirm that first.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Version: Development branch (https://dev.graphite.art/)
Browser: Firefox
Personally not sure if this is a bug or a UX problem. Here is a video recording:
https://github.com/user-attachments/assets/29053c91-5dbc-4204-a07d-f5b112a63977
The same issue applies to the stroke as well.
Upon inspection of the node graph to see where it was getting the colour from:
I found the reason is that the merge node prefers the bottom input's fill and stroke colour over the original top input's fill and stroke colour, which isn't changed when changing the colour of the stroke or fill through the method shown in the video. Forcing the user to either move up the fill and stroke manually, or disabling and enabling the fill and stroke so it is forced pasted the merge node in the order.
If this is a duplicate, I misunderstood something, or someone thinks of a more accurate title please let me know. (feel free to ping me on discord as I am more likely to see it.)
- Ngôn ngữ chính
- Rust
- Star
- 27.8k
- Fork
- 1.3k
- Merge trung bình
- 10 giờ 53 phút
- Pull request đã merge (30 ngày)
- 195
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của GraphiteEditor/Graphite
-
`third-party-licenses` fails on Windows when the parent process is PowerShellCó thể đã có người làm @sahilshingate01 đã nhận 11 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
GraphiteEditor/Graphite#4745 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Resizing a text box by one edge handle incorrectly locks both Max Width and Max Height, silently clipping textCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
GraphiteEditor/Graphite#4396 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Type hint is selected in node graph viewCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
GraphiteEditor/Graphite#4311 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
`cargo run` fails on WebAssembly target due to `CARGO_BUILD_TARGET` leaking into `rustc_codegen_spirv` buildCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
GraphiteEditor/Graphite#3939 · 2 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
GraphiteEditor/Graphite#4901 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của GraphiteEditor/Graphite
Issue tương tự
-
question
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
autoindex tests: denied loopback bind shows only PermissionDenied (255 failures), not the causeĐang mởfield-report
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Define the ypass-core error typeĐang mởarea:core
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
julian-hemmer/ypass#4 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
davidkodar/klipp#3426 ·
Maintainer thường phản hồi trong vòng 1 ngày