TabPageViewController icon indicating copy to clipboard operation
TabPageViewController copied to clipboard

Add setTabItemTitle

Open isseium opened this issue 7 years ago • 4 comments

Sorry for writing in Japanese.

立て続けに失礼します。:)

TabPageViewController 表示後に、タブのタイトルを変更したく setTabItemTitle(_ title: String, at: Int) というメソッドを用意しました。

もともと当方のfork先で類似の変更をしていたのですが、本家に取り込んでいただいたほうがメリットがあると判断しましたので、問題がなければ取り込んでいただけないでしょうか。

すでの現状のコードで変更する方法があればご教示ください。

よろしくお願いします。

====

Automatic Translation using Google Translate

After displaying TabPageViewController, we have prepared a method called setTabItemTitle that allows you to change the title of the tab.

Originally we made a similar change at our fork destination, but we decided that it is beneficial to incorporate it in our headquarters, so if you do not have a problem, would you please incorporate it?

Please teach if there is any way to change with the current code.

Thank you.

isseium avatar Apr 13 '17 02:04 isseium

どういったときに必要になるのでしょうか? ユースケースが思いつかず、ライブラリに組み込むか悩んでいます。

EndouMari avatar Apr 14 '17 03:04 EndouMari

ご連絡ありがとうございます。

使用例ですが、検索結果の件数をタブに表示したいというモチベーションから提案しております。

例として、ニュースアプリを想定していただきたいです。 ニュースには、「政治」「経済」「エンタメ」というカテゴリがあります。

この3つのカテゴリを横断して任意のワードで絞り込みをする検索機能を実装します。 その場合に、検索結果で、各カテゴリごとにタブを用意し、「政治(3)」「経済(10)」「エンタメ(0)」のようにヒット件数をタブに表示したく、タブのタイトルを変更するメソッドが欲しいです。

わかりにくければ補足いたしますのでご指摘ください。

また、もし方針にあわずライブラリへの組み込みが難しい場合は、 tabView (より局所的にいうと tabView.pageTabItems) へのアクセス出来る方法を検討いただけないでしょうか。

よろしくお願いします。

isseium avatar Apr 14 '17 06:04 isseium

使用例を理解しました。 ありがとうございます! とてもいい提案だと思います。

使う人によってはまた別なカスタマイズがしたいとおもうので 今頂いているPRを取り込むのではなく UITableViewDataSource,Delegateなどのような形で提供できれば良いのかなとおもいました。

実装に時間がかかると思うのですぐに対応するのはむずかしいのですが、 いかがでしょうか?

EndouMari avatar Apr 14 '17 06:04 EndouMari

ありがとうございます。

もし DataSource や Delegate のほうがエレガントにタイトルを変更する方法が提供されるのであればそちらでぜひお願いします。

正式に対応いただくまでは、現在PRしているものを自前のリポジトリで管理して対応したいと思います。 (急かすわけではありませんが、なるべく早いとうれしいですw :)

isseium avatar Apr 14 '17 07:04 isseium