Issue with visible content / and contentSizes after orientation change about colormatchtabs HOT 8 CLOSED

Quick workaround:

ScrollMenu.swift

` override public func layoutSubviews() {
super.layoutSubviews()

    layoutContent()
    let contentOffsetX = bounds.width * CGFloat(currentPage)
    setContentOffset(CGPoint(x: contentOffsetX, y: contentOffset.y), animated: false)
}`

We need to store 'currentPage' - maybe good idea to make it public.

@serejahh: Feedback? :)

from colormatchtabs.

@serejahh: Oh didn't see your fix. Thought you just closed the issue.

Your latest fix caused wrong index after rotation.

1. Portrait - Index 2 selected
2. Rotate
3. Landscape - Index 2 still selected
4. Rotate
5. Portrait - Index 3 selected !!!

Haven't seen the issue with my workaround though... Going to test a lil bit more

from colormatchtabs.

Please check if your fix works better. Then I'll merge it

from colormatchtabs.

It seems it works, but when you tap on a tabbar item, a page doesn't change and you can't swipe

from colormatchtabs.

yes you are correct - swipe is broken with my workaround but correct index is shown :(

from colormatchtabs.

public var indexOfVisibleItem: Int {
    if bounds.width > 0 {
        return Int(round(contentOffset.x / bounds.width))
    }
    return 0
}

is returning wrong index after 2nd orientation change. So 'previousIndex' will be messed up.

Maybe related to "Out Of Range" bug ?

from colormatchtabs.

Seems to be true. When a device is just been rotated from landscape to portrait, bounds is for portrait, but contentSize and contentOffset are for landscape

from colormatchtabs.

Still getting issue in landscape mode

from colormatchtabs.