Deprecation of `DiscreteSurface` is broken

[haakon-e]

On Makie v0.21.11 I can define methods that dispatch on Makie.DiscreteSurface

julia> f(::Makie.DiscreteSurface) = 1
f (generic function with 1 method)

however on Makie v0.21.12 I get the following error:

julia> f(::Makie.DiscreteSurface) = 1
ERROR: ArgumentError: invalid type for argument number 1 in method definition for f at REPL[9]:1
Stacktrace:
 [1] top-level scope
   @ REPL[9]:1

I suspect this comes from #4333, which deletes https://github.com/MakieOrg/Makie.jl/blob/65a718195a30fc4f3775f5457357e1ba35248745/src/deprecated.jl#L4-L5 and replaces it with https://github.com/MakieOrg/Makie.jl/blob/b971fbb84357ba1e66a17f66b50dda86bea9c8a4/src/deprecated.jl#L5-L13

so that DiscreteSurface is no longer recognized as a DataType, but instead is a Function.

---

• [x] what version of Makie are you running? (]st -m Makie) 0.21.12
• [x] can you reproduce the bug with a fresh environment ? (]activate --temp; add Makie)
• [x] What platform + GPU are you on? Mac M1

[SimonDanisch]

Hm, Yeah I guess that's the disadvantage of that approach. We could make it a type as well, or say this has been breaking for multiple versions, so it's fine to leave it broken.

[haakon-e]

[...] or say this has been breaking for multiple versions, so it's fine to leave it broken.

What do you mean by this? 0.21.12 was released two days ago, so "breaking for multiple versions" doesn't seem accurate; or do you mean it has been deprecated for multiple versions?

[ffreyer]

DiscreteSurface has been deprecated for ~10 months now, since version 0.20