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

IOS-9433 [iOS]Mistica Implementing the Small style in the Header component #316

Conversation

DavidMarinCalleja
Copy link
Contributor

@DavidMarinCalleja DavidMarinCalleja commented Oct 19, 2023

🎟️ Jira ticket

IOS-9433 [iOS]Mistica Implementing the Small style in the Header component

πŸ₯… What's the goal?

A new style called Small has been added to the Header component.
Header Components Specs

🚧 How do we do it?

I've added two new styles to HeaderViewStyle.swift

public enum HeaderViewStyle {
    case normal
    case normalSmall.    // <-- NEW
    case inverse
    case inverseSmall    // <-- NEW
}

πŸ§ͺ How can I verify this?

SwiftUI UIKit
normalSmall inverseSmall

πŸ‘ AppCenter build

You can use the alpha build https://install.appcenter.ms/orgs/Tuenti-Organization/apps/Mistica-SwiftUI-iOS
Screenshot 2023-10-20 at 10 53 47

Version 26.2.0 (1)
Oct 20, 2023 at 10:44
4.71 MB

@github-actions
Copy link

github-actions bot commented Oct 19, 2023

Screenshot tests report

βœ”οΈ All passing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@WanaldinoTelefonica WanaldinoTelefonica left a comment

Choose a reason for hiding this comment

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

πŸŽ†


show(vc, sender: self)
}
}

private extension UICatalogHeaderViewController {
func createSegmentedControl(reuseIdentifier: String, firstSegmentedTitle: String, secondSegmentedTitle: String) -> UISegmentedControlTableViewCell {
func createSegmentedControl(reuseIdentifier: String, segments: String...) -> UISegmentedControlTableViewCell {
Copy link
Contributor

Choose a reason for hiding this comment

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

❀️

Copy link
Contributor

@dhidalgofadrique dhidalgofadrique left a comment

Choose a reason for hiding this comment

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

πŸ‘

@DavidMarinCalleja DavidMarinCalleja merged commit 281be43 into main Oct 25, 2023
3 checks passed
@DavidMarinCalleja DavidMarinCalleja deleted the IOS-9433-Implementing-the-Small-style-in-the-Header-component branch October 25, 2023 07:51
@tuentisre
Copy link
Collaborator

πŸŽ‰ This PR is included in version 26.3.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants