David Peck

Performance issues

4

I've narrowed down a performance issue we're having with the url picker. When we request a Link, it's taking about 7ms to return the data. `content.Link.FirstOrDefault(l => l.Url != null)`...

Properties with the aliases that match keywords fail

3

One of my content type property aliases is called 'abstract'. While potentially short sighted this makes sense from an article perspective but as you're using the alias when building the...

Fixes your #80

1

Adding a properties property prevents the anchor modal from failing

Caching SVG doesn't cache CSS class name and Viewbox settings

1

There is an issue where caching an SVG with a class name (or Viewbox setting) causes the class name to be contained in the cache. If outputting the same SVG...

our-svg caches the CSS class

3

For the same SVG I have both ``` ``` With the cache turned on the class attribute is cached and 'foo' is rendered in both locations. I suggest adding the...

our-svg default cache can't be overridden

1

If you set the SVG cache to be on in appsettings.config, and then wish to disable it on a specific tag instance then it is not possible. Ideally `InlineSvgTagHelper.Cache` (https://github.com/umbraco-community/Our-Umbraco-TagHelpers/blob/main/Our.Umbraco.TagHelpers/InlineSvgTagHelper.cs#L72)...

Idea: our-text-if

2

Rather than have: ``` @Model.Heading ``` you can just have ``` ```

Data retention

2

Data Protection Officers should have a data retention policy which states that old data is deleted after a period of time. It is likely that PII held within order will...

Simply an exception catch around the HTML parsing. Logs the error as a warning and returns an empty string.