Refactor: GymDetailViewController Sections
@kevinchan159 is already working on this.
Since Nov 10, 2019.
Assessment
This issue has not been assessed yet.
Description
Currently, inside GymDetailViewController we define an enum Section follows:
// MARK: - Private classes/enums
private struct Section {
var items: [ItemType]
}
and then define var sections: [Section] to just be:
self.sections = [
Section(items: [.hours, .busyTimes, .facilities, .classes([])])
]
This means that what we end up with is just one giant section with 4 cells.
We should change this so that we have 4 sections, with one cell in each. This is will help reduce with the increasingly nested view hierarchy we have
- Dominant language
- Swift
- Stars
- 18
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from cuappdev/uplift-ios
-
Type: Maintenance
Difficulty 4/5 3-5 days Newbie friendliness 35/100
cuappdev/uplift-ios#266 ·
-
Type: Refactor
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
cuappdev/uplift-ios#233 ·
-
Type: Refactor
Difficulty 4/5 3-5 days Newbie friendliness 25/100
cuappdev/uplift-ios#232 ·
-
Type: Refactor
cuappdev/uplift-ios#214 · 1 assignee ·
-
Priority: High
cuappdev/uplift-ios#203 · 1 assignee ·
All issues in cuappdev/uplift-ios
Similar issues
-
task
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
vsanthanam/JBird#429 ·
-
needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
nightscout/Trio#1574 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
pending-maintainer-response pending-triage
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
area:build bug good first issue P2
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
uttrflow/uttrflow-swift#1412 ·