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

[MBL-902] [MBL-934] [MBL-928] Use of AI Screen, Analytics, Question Headers, Translations #1843

Merged
merged 27 commits into from
Aug 22, 2023
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
bfde6bc
Updated model with ai disclosure fields
msadoon Aug 2, 2023
bb2c048
added use of ai tab
msadoon Aug 8, 2023
32dd553
Merge branch 'main' into mbl-900/new-tab-use-of-ai
msadoon Aug 9, 2023
b879f74
Merge branch 'main' into mbl-900/new-tab-use-of-ai
msadoon Aug 9, 2023
f913b88
fixed a test for use of ai tab title
msadoon Aug 9, 2023
e32af42
Merge branch 'mbl-900/new-tab-use-of-ai' of https://github.com/kickst…
msadoon Aug 9, 2023
ad87197
formatting
msadoon Aug 9, 2023
f10b5dd
updated all cells for use of ai tab except the funding ones. Up next.
msadoon Aug 9, 2023
aafa8a2
added final cell and tested - pretty close to design, one padding iss…
msadoon Aug 10, 2023
7bb8164
Merge branch 'main' into mbl-902/use-of-ai-screen
msadoon Aug 15, 2023
590d75f
Stop using stack view in order to align checkmark and label
ifosli Aug 15, 2023
0d66016
Update footer to link to AI policy
ifosli Aug 16, 2023
fecc868
Update link to have plain text default
ifosli Aug 17, 2023
06b15b2
Merge branch 'main' into mbl-902/use-of-ai-screen
msadoon Aug 21, 2023
de7f8b2
Use translatable strings
ifosli Aug 21, 2023
b84815b
added analytics, checking if involves ai is true along with the exist…
msadoon Aug 21, 2023
90764de
Merge branch 'mbl-902/use-of-ai-screen' of https://github.com/kicksta…
msadoon Aug 21, 2023
15e8a20
Add a11y label so voiceover reads tab correctly
ifosli Aug 22, 2023
84b77b1
Added titles to questions and answers, all sections should be accurat…
msadoon Aug 22, 2023
55736ad
Merge branch 'mbl-902/use-of-ai-screen' of https://github.com/kicksta…
msadoon Aug 22, 2023
c433f53
correct to title headers and order
msadoon Aug 22, 2023
2b17624
tested kickstarter-ios and library
msadoon Aug 22, 2023
2a0559f
ksapi changes tested
msadoon Aug 22, 2023
f657b3c
Merge branch 'main' into mbl-902/use-of-ai-screen
msadoon Aug 22, 2023
923813b
pr comments
msadoon Aug 22, 2023
9cfd437
Merge branch 'mbl-902/use-of-ai-screen' of https://github.com/kicksta…
msadoon Aug 22, 2023
7973635
removing cell separator and added a FIXME for title cell
msadoon Aug 22, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ final class ProjectTabCheckmarkListCell: UITableViewCell, ValueCell {
self?.fundingStackView.arrangedSubviews.forEach { $0.removeFromSuperview() }

_ = options.map { [weak self] optionText in
let imageLabelStackView = UIStackView(frame: .zero)
|> imageLabelStackViewStyle
let imageLabelView = UIView(frame: .zero)

let label = UILabel(frame: .zero)
|> \.text .~ optionText
Expand All @@ -60,10 +59,19 @@ final class ProjectTabCheckmarkListCell: UITableViewCell, ValueCell {
let icon = UIImageView(frame: .zero)
|> iconImageStyle

imageLabelStackView.addArrangedSubview(icon)
imageLabelStackView.addArrangedSubview(label)
imageLabelView.addSubview(icon)
imageLabelView.addSubview(label)

self?.fundingStackView.addArrangedSubview(imageLabelStackView)
NSLayoutConstraint.activate([
msadoon marked this conversation as resolved.
Show resolved Hide resolved
icon.leadingAnchor.constraint(equalTo: imageLabelView.leadingAnchor),
icon.topAnchor.constraint(equalTo: imageLabelView.topAnchor, constant: Styles.grid(1)),
label.leadingAnchor.constraint(equalTo: icon.trailingAnchor, constant: Styles.grid(2)),
label.topAnchor.constraint(equalTo: imageLabelView.topAnchor),
label.trailingAnchor.constraint(equalTo: imageLabelView.trailingAnchor),
label.bottomAnchor.constraint(equalTo: imageLabelView.bottomAnchor)
])

self?.fundingStackView.addArrangedSubview(imageLabelView)
}

self?.fundingStackView.setNeedsDisplay()
Expand Down Expand Up @@ -136,15 +144,6 @@ private let fundingStackViewStyle: StackViewStyle = { stackView in
|> \.spacing .~ Styles.grid(2)
}

private let imageLabelStackViewStyle: StackViewStyle = { stackView in
stackView
|> \.axis .~ .horizontal
|> \.isLayoutMarginsRelativeArrangement .~ true
|> \.layoutMargins .~ UIEdgeInsets(all: Styles.grid(0))
|> \.spacing .~ Styles.grid(2)
|> \.alignment .~ .top
}

private let iconImageStyle: ImageViewStyle = { imageView in
imageView
|> \.tintColor .~ .ksr_create_700
Expand All @@ -154,6 +153,7 @@ private let iconImageStyle: ImageViewStyle = { imageView in
|> UIImageView.lens.contentHuggingPriority(for: .horizontal) .~ .defaultLow
|> UIImageView.lens.contentCompressionResistancePriority(for: .vertical) .~ .required
|> UIImageView.lens.contentCompressionResistancePriority(for: .horizontal) .~ .required
|> \.translatesAutoresizingMaskIntoConstraints .~ false
}

private let optionTextLabelStyle: LabelStyle = { label in
Expand All @@ -162,4 +162,5 @@ private let optionTextLabelStyle: LabelStyle = { label in
|> \.font .~ UIFont.ksr_body()
|> \.numberOfLines .~ 0
|> \.textColor .~ .ksr_support_700
|> \.translatesAutoresizingMaskIntoConstraints .~ false
}