fill patterns should be passed by object, not by name as string

[boeckhoff]

This way they can be loaded dynamically when needed.

this would require an id check each time which checks if the pattern is already loaded.

Advantages are:

• better performance because only needed patterns are loaded

• consistency with symbol-shape tag