-
Notifications
You must be signed in to change notification settings - Fork 7
[PLANPRO-5832] Signal ausleger with foundation #2047
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
d68dac0
048ac5b
8aba073
26f709b
3e6c25f
37ada06
7247318
8b76ea5
146ceba
302508c
c102435
8a4fb50
001cc34
f2263ee
1aae3b0
aa4db18
fb3c896
233c979
00c69c3
6d975cb
27799a1
92a8fb7
b317be5
78ef269
44833e5
b21f36a
7b1fab3
448ef5c
584c641
e388ebe
a2538ae
6972849
2e504e1
e5e1790
e66fda9
e6b12ca
778893d
9bd990d
06008d0
ab2e56a
913e06b
dde8e22
faf9daa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
JanErikVoigt marked this conversation as resolved.
Show resolved
Hide resolved
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,11 +26,13 @@ export default abstract class AbstractDrawSVG { | |
| } | ||
|
|
||
| /** | ||
| * Create a Svg for a feature | ||
| * Create a Svg for a feature. Return null, if | ||
| * this Drawer cannot draw that feature or if it is faulty. | ||
| * If all DrawSVGs return null, an error is displayed. | ||
| * @param data feature data | ||
| * @param label {@link Label} | ||
| */ | ||
| public abstract drawSVG<T extends object>(data: T, label?: Label): ISvgElement | ||
| public abstract drawSVG<T extends object>(data: T, label?: Label): ISvgElement | null | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. instead of
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I made SvgDrawBridge an AbstractDrawSVG. Now, there are two for Signals... (SvgDrawSignal and SvgDrawBridge). Don't you think it is sensible to model it like this? Now flow is:
|
||
|
|
||
| public getSvgFromCatalog<T extends object> (data: T): ISvgElement | null { | ||
| return this.catalogService.getObjSvg(data, this.getFeatureType()) | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.