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 extension of UIlabel for adding line spacing with the method of n… #438

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 10 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,14 @@ All notable changes to this project will be documented in this file.

## Unreleased

**UILabelExtensions**
Copy link
Collaborator

Choose a reason for hiding this comment

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

Minor : With a number (in this case 1).

Copy link
Author

Choose a reason for hiding this comment

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

So do I need removed this spacing ??

Copy link
Collaborator

Choose a reason for hiding this comment

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

have a 1 prior to that. Like a numbering thing. But that is really cosmetic, I would suggest taking a look at writing unit tests for the same.

- `setLineSpacing(lineSpacing : CGFloat)` in [[PR]](https://github.com/goktugyil/EZSwiftExtensions/pull/438) by *osama10*


1. **Double**
- `**?(lhs:Double, rhs:Double)` in [[PR]](https://github.com/goktugyil/EZSwiftExtensions/pull/443) by *Khalian*


## [Release 1.10]

### New platform added.
Expand Down Expand Up @@ -57,7 +62,7 @@ All notable changes to this project will be documented in this file.
- `validateEmail()` in [[PR]](https://github.com/goktugyil/EZSwiftExtensions/pull/429) by *Vic-L*
- `validateDigits()` in [[PR]](https://github.com/goktugyil/EZSwiftExtensions/pull/429) by *Vic-L*

### Deprecated/Renamed extensions
### Deprecated/Renamed extensions
1. **UIViewController**
- `public func hideKeyboardWhenTappedAround(cancelTouches: Bool = false)` in [[PR]](https://github.com/goktugyil/EZSwiftExtensions/pull/395) by *lfarah*

Expand Down
83 changes: 59 additions & 24 deletions EZSwiftExtensionsTests/UILabelTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@
//

#if os(iOS) || os(tvOS)
<<<<<<< HEAD
Copy link
Collaborator

Choose a reason for hiding this comment

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

You have unresolved merge commits here.

=======

import XCTest
@testable import EZSwiftExtensions
Expand All @@ -22,34 +24,67 @@ class UILabelTests: XCTestCase {
XCTAssertEqual(label2.font.pointSize, 20)
XCTAssertEqual(label.font.pointSize, 17)
}
>>>>>>> 638c47200b16776d978dc4598237109396915878
Copy link
Collaborator

Choose a reason for hiding this comment

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

This looks like an unresolved merge, please collapse your commits into one idempotent change.

I cannot even begin to tell you how much I despise non fast forward merges. Rebase your changes and replay them on top of the current HEAD commit.


func testSet() {
import XCTest
@testable import EZSwiftExtensions
class UILabelTests: XCTestCase {

let label = UILabel(frame: CGRect(x: 0, y: 0, width: 200, height: 50))
label.set(text: "EZSwiftExtensions✅", duration: 1)
XCTAssertEqual(label.text, "EZSwiftExtensions✅")
func testInit() {

let label = UILabel(x: 0, y: 0, w: 200, h: 50)
let expected = UILabel(frame: CGRect(x: 0, y: 0, width: 200, height: 50))
let label2 = UILabel(x: 0, y: 0, w: 200, h: 50, fontSize: 20)

XCTAssertEqual(label.frame, expected.frame)
XCTAssertEqual(label2.font.pointSize, 20)
}

label.text = ""
label.set(text: "EZSwiftExtensions🚀", duration: 0)
XCTAssertEqual(label.text, "EZSwiftExtensions🚀")
func testSet() {

let label = UILabel(frame: CGRect(x: 0, y: 0, width: 200, height: 50))
label.set(text: "EZSwiftExtensions✅", duration: 1)
XCTAssertEqual(label.text, "EZSwiftExtensions✅")

label.text = ""
label.set(text: "EZSwiftExtensions🚀", duration: 0)
XCTAssertEqual(label.text, "EZSwiftExtensions🚀")

label.text = ""
label.set(text: "EZSwiftExtensions❤️", duration: 1)
XCTAssertEqual(label.text, "EZSwiftExtensions❤️")
}

label.text = ""
label.set(text: "EZSwiftExtensions❤️", duration: 1)
XCTAssertEqual(label.text, "EZSwiftExtensions❤️")
}

var waitExpectation: XCTestExpectation?

func wait(duration: TimeInterval) {
waitExpectation = expectation(description: "wait")
Timer.scheduledTimer(timeInterval: duration, target: self,
selector: #selector(UILabelTests.onTimer), userInfo: nil, repeats: false)
waitForExpectations(timeout: duration + 3, handler: nil)
func testSetLineSpacing(){

let textForTesting = "I am testing test Set line spacing method"
var paragraphStyle : NSMutableParagraphStyle?

Copy link
Collaborator

Choose a reason for hiding this comment

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

nitpick : too many empty lines here are there that do not serve a purpose.

let label = UILabel(frame: CGRect(x: 0, y: 0, width: 200, height: 50))
label.text = textForTesting
label.setLineSpacing(lineSpacing: 1.5)

label.attributedText?.enumerateAttribute(NSParagraphStyleAttributeName , in: NSMakeRange(0, (label.attributedText?.length)!), options: [.longestEffectiveRangeNotRequired]) { value, range, isStop in
if let value = value {
paragraphStyle = value as NSMutableParagraphStyle
}
}

XCTAssertEqual(paragraphStyle?.lineHeightMultiple, 1.5)
}

var waitExpectation: XCTestExpectation?

func wait(duration: TimeInterval) {
waitExpectation = expectation(description: "wait")
Timer.scheduledTimer(timeInterval: duration, target: self,
selector: #selector(UILabelTests.onTimer), userInfo: nil, repeats: false)
waitForExpectations(timeout: duration + 3, handler: nil)
}

func onTimer() {
waitExpectation?.fulfill()
}
}

func onTimer() {
waitExpectation?.fulfill()
}
}

#endif
13 changes: 13 additions & 0 deletions Sources/UILabelExtensions.swift
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,19 @@ extension UILabel {
self.text = _text
}, completion: nil)
}

// Set lineSpacing for UILabel
public func setLineSpacing(lineSpacing: CGFloat) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it possible to write a unit test for this ?

let paragraphStyle = NSMutableParagraphStyle()
paragraphStyle.lineSpacing = 1.0
paragraphStyle.lineHeightMultiple = lineSpacing
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is confusing. Does this mean your spacing = 1.0 * lineHeight = lineSpacing. If so, you could just call the method arg as lineHeightMultiple.

paragraphStyle.alignment = self.textAlignment

let attrString = NSMutableAttributedString(string: self.text!)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This could actually live as its own var. Otherwise its unnecessary allocation of the attr string per call.

attrString.addAttribute(NSFontAttributeName, value: self.font, range: NSMakeRange(0, attrString.length))
Copy link
Collaborator

Choose a reason for hiding this comment

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

NSMakeRange is repeated, you can abstract that away as a single call.

attrString.addAttribute(NSParagraphStyleAttributeName, value:paragraphStyle, range:NSMakeRange(0, attrString.length))
self.attributedText = attrString
}
}

#endif