-
-
Notifications
You must be signed in to change notification settings - Fork 117
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Refine tags to a more specific type There is a regression introduced during the Scala 3 upgrade where the `scalatags.Text.all.doctype` case class no longer worked with the pre-existing tags since its second argument accepts a `scalatags.text.Frag` but the tags themselves refined to `<tag-name>.Self` (the <tag-name> is a placeholder for the tag's name). As it turns out, this self type was not being refined by the tags themselves because for some reason, the Scala 3 compiler resolves this type to be `scalatags.generic.TypedTag` i.e. it resolves to the more generic type as opposed to the more specific type of `scalatags.Text.TypedTag`. Fortunately in Scala 2, this works out fine as the type does get refined and therefore the tags can be used inside a `scalatags.text.Frag`. This commit essentially adds type annotations to all the tags under the text package so that the more specific `ConcreteHtmlTag[String]` is used instead of the more generic `TypedTag`. * Add doctype compilation test and fix extraneous warnings.
- Loading branch information
1 parent
2d4454c
commit 47f8f7f
Showing
6 changed files
with
193 additions
and
189 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.