search

hotwired / hotwire-native-ios

Hotwire Native for iOS
MIT License
73 stars 7 forks source link

Naming question #3

Closed joemasilotti closed 11 months ago

joemasilotti commented 11 months ago

@jayohms, we spoke about this so I wanted to double check.

I'm using the Web* prefix for the demo app. Which subclasses from a Hotwire* class in the actual repo. Does that make sense to you? Or do you want the repo itself to use the Web* prefix?

https://github.com/hotwired/hotwire-native-ios/blob/4b636f2bbf6c835d38f32efca47f016346b1a942/Demo/WebViewController.swift#L4

jayohms commented 11 months ago

Yep, that makes sense to me 👍 And ideally, we won't even need most subclasses in the demo app, if we can find ways to deliver a good set of default (customizable) functionality in the Hotwire* classes.

joemasilotti commented 11 months ago

Agreed. I took a pass in #4. Let me know if that's what you were thinking!

jayohms commented 11 months ago

Nice 👍. I don't have fully formed thoughts on configuration vs customization yet, so let's see how it all plays out when we see it all together.

joemasilotti commented 11 months ago

Cool, thanks Jay! I'll get everything ready in the next few days and present the full repo for review from the team.