Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Creator/Custom Design Items #189

Merged
merged 17 commits into from May 27, 2020
Merged

Add Creator/Custom Design Items #189

merged 17 commits into from May 27, 2020

Conversation

otaviocc
Copy link
Contributor

@otaviocc otaviocc commented May 24, 2020

Description

There are so many creators and custom designs to track and try that sometimes it's hard to keep track of them. A good place to store these information would be in this companion app.

Implementation

The main types added by this commit are:

  • Design: simple model with title, code, and description.
  • DesignRowView and DesignRowViewModel: view and view model used to populate the table.
  • DesignFormView and DesignFormViewModel: view and view model used to add a new design to the list.

This commit also touches the caching layer, adding designs as an optional array.

Screenshots

screenshots

Demo

output

In case of bugs, I can help Blathers on that.

@otaviocc otaviocc changed the title Add Creator/Custom Items Add Creator/Custom Design Items May 24, 2020
@Dimillian
Copy link
Owner

Dimillian commented May 25, 2020

Hello, love the feature. But in the collection screen, this segmented control will crop the label on top. If you try on an iPhone 8 for example. Or a landscape iPad. Maybe it's not the right place to be or maybe we have to rework the top selector to something custom. I'll review the code shortly.

@otaviocc
Copy link
Contributor Author

Hello, love the feature. But in the collection screen, this segmented control will crop the label on top. If you try on an iPhone 8 for example. Or a landscape iPad. Maybe it's not the right place to be or maybe we have to rework the top selector to something custom. I'll review the code shortly.

Oops :-) Do you have any suggestion for the ui/ux, @Dimillian? I can give it a try.

@otaviocc
Copy link
Contributor Author

Hello, love the feature. But in the collection screen, this segmented control will crop the label on top. If you try on an iPhone 8 for example. Or a landscape iPad. Maybe it's not the right place to be or maybe we have to rework the top selector to something custom. I'll review the code shortly.

I have suggestion and will implement and propose it later tonight.

@Dimillian
Copy link
Owner

Yes sorry didn't had time to think about it yet. Can't wait to see your proposition.

@renaudjenny
Copy link
Collaborator

What about having a More tab which directs to a list of "other" available collections.
Screenshot 2020-05-26 at 18 46 20
Screenshot 2020-05-26 at 18 46 54

@Dimillian
Copy link
Owner

Dimillian commented May 26, 2020

I like it. I would move critters into the more tab and leave the custom lists as it is. We already have a whole critters section in the dashboard. And List have a subscribe call :p

## Description

Sometimes it's hard to keep track of creators
and items we want to follow or clothes and
patterns we want to use.

A good place to track these information would be
in the companion application.

## Implementation

The main types added by this commit are:

- `Design`: simple model with title, code,
   and description.
- `DesignRowView` and `DesignRowViewModel`: view
   and view model used tp populate the table
- `DesignFormView` and `DesignFormViewModel`: view
   and view model used to add a new design to
   the list.

This commit also touches the caching layer, adding
designs as an optional array.
# Conflicts:
#	ACHNBrowserUI/ACHNBrowserUI/views/shared/Sheet.swift
# Conflicts:
#	ACHNBrowserUI/ACHNBrowserUI/views/shared/Sheet.swift
# Conflicts:
#	ACHNBrowserUI/ACHNBrowserUI/views/userLists/UserListDetailView.swift
@otaviocc
Copy link
Contributor Author

new

I've added the "More" tab and moved Critters and Designs there. Let me know what you think, @Dimillian, @renaudjenny.

@Dimillian
Copy link
Owner

Awesome !!!! I guess this will land in master very soon :)

@renaudjenny
Copy link
Collaborator

@otaviocc awesome! Can't wait to have this on the app.
I quickly checked the code, and it's looking good!

List {
CollectionMoreDetailView(viewModel: CollectionMoreDetailViewModel())
}
.previewLayout(.sizeThatFits)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's smart. I've never think about this one!

@Dimillian
Copy link
Owner

Making few fixes for localization and merging, great works!

@Dimillian Dimillian merged commit 9d33500 into Dimillian:master May 27, 2020
@otaviocc otaviocc deleted the designs branch May 27, 2020 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants