Issue 67765

Summary: custom shape extrusion surface doesn't work
Product: Draw Reporter: Regina Henschel <rb.henschel>
Component: editingAssignee: AOO issues mailing list <issues>
Status: ACCEPTED --- QA Contact:
Severity: Trivial    
Priority: P3 CC: issues, mseidel, zhuangyuelin
Version: 680m176   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
See Also: https://bz.apache.org/ooo/show_bug.cgi?id=98334
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
Suggestions how to solve the problem none

Description Regina Henschel 2006-07-25 17:57:37 UTC 
Take a Draw document and draw a custom shape.
Extrude it and increase the extrusion depth.
Now open toolbar "surface". You find the entries "Wire Frame", "Matt",
"Plastic", "Metal". Try them all.
Expected: The specular light will increase from "Matt" to "Metal"
Seen behavior: There is no difference between "Matt", "Plastic" and "Metal".
Comment 1 wolframgarten 2006-07-26 07:20:08 UTC 
Reproducible. Reassigned.
Comment 2 groucho266 2006-07-26 08:35:36 UTC 
Please take over.
Comment 3 sven.jacobi 2006-08-16 16:47:26 UTC 
accepted
Comment 4 Martin Hollmichel 2007-11-09 17:01:45 UTC 
set target from 2.x to 3.x according
http://wiki.services.openoffice.org/wiki/Target_3x
Comment 5 Regina Henschel 2021-11-06 14:59:54 UTC 
Created attachment 87071 [details]
Suggestions how to solve the problem

The attached file contains my patch for LibreOffice. Because the relevant code was not changed significantly by LibreOffice, it is likely that you can use it nearly directly.

Some comments on the patch are in https://gerrit.libreoffice.org/c/core/+/124720
and more comments about the patch are in https://bugs.documentfoundation.org/show_bug.cgi?id=140321#c7

I give the code under Apache License, Version 2.0 to you.
Comment 6 Matthias Seidel 2021-11-06 20:18:14 UTC 
Thanks Regina!

This is more complex than the last one but I will have a look into it.
Comment 7 Regina Henschel 2021-11-06 21:12:25 UTC 
(In reply to Matthias Seidel from comment #6)
> Thanks Regina!
> 
> This is more complex than the last one but I will have a look into it.

It is indeed complex. But I thought it would be useful to attach it here, so that if someone will fix this bug, he can test my ideas without worrying about the license.
Comment 8 Matthias Seidel 2021-11-06 21:26:25 UTC 
*** Issue 98334 has been marked as a duplicate of this issue. ***
Comment 9 Regina Henschel 2022-02-10 23:08:17 UTC 
There are more problems than I first thought. The current lighting has principle errors. For details see https://gerrit.libreoffice.org/c/core/+/128449. It has lot of comments, at least in patch set #10.

You can use the patch from https://gerrit.libreoffice.org/c/core/+/128449 under Apache License, Version 2.0.

In case someone will work on this and has questions about the code, please drop me a mail.