You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A quick note that while upgrading from 1.x to 2.x one change I had to address was that a lot of items that were previously available in the properties module (such as Script and BidiClass) are now in properties::props. Unless there's some compelling reason I'm missing, I think it makes sense to have these items available in the root properties module? It would make the upgrade experience cleaner for folks, and also provide a generally cleaner API.
The text was updated successfully, but these errors were encountered:
This was discussed in #3573 and implemented in #5548
In general it makes the crate more organized for all property enums to be in a module together props, since icu_properties exports a bunch of other types for reading the properties. It is nice to see a list in one place.
A quick note that while upgrading from 1.x to 2.x one change I had to address was that a lot of items that were previously available in the
properties
module (such asScript
andBidiClass
) are now inproperties::props
. Unless there's some compelling reason I'm missing, I think it makes sense to have these items available in the rootproperties
module? It would make the upgrade experience cleaner for folks, and also provide a generally cleaner API.The text was updated successfully, but these errors were encountered: