Skip to content

Conversation

@dhyash-simform
Copy link
Collaborator

Description

Checklist

  • The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
  • I have followed the [Contributor Guide] when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

settings: ChatSettings(
muteStatus: MuteStatus.muted,
pinStatus: PinStatus.pinned,
pinTime: DateTime.now().subtract(const Duration(minutes: 5)),
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can use fiveMinAgo here.

_chatListStreamController.sink.add(initialChatMap);
}

void updateChatSettings(
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe this can be achieved with updateChat method as well.

Comment on lines 11 to 12
final double? iconSize;
final Color? iconColor;
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can drop icon prefix in the field name since that's included in the class' name.

@dhyash-simform dhyash-simform force-pushed the feat/chat_list_pin_mute_icons branch 7 times, most recently from b4ac82b to 4c9501a Compare August 1, 2025 10:27
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
import 'package:chatview/src/values/typedefs.dart';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use relative import here.

@dhyash-simform dhyash-simform force-pushed the feat/chat_list_pin_mute_icons branch 3 times, most recently from 34640e8 to 5d34f60 Compare August 19, 2025 12:04
@dhyash-simform dhyash-simform changed the base branch from main to feat/chat_list_ui_base August 19, 2025 12:04
- Provided timestamp in last message time builder
- Provided chat data in profile avatar, chat & trailing builders
@dhyash-simform dhyash-simform force-pushed the feat/chat_list_pin_mute_icons branch from 5d34f60 to 7e081d0 Compare August 20, 2025 08:04
@aditya-css aditya-css merged commit e2428ce into feat/chat_list_ui_base Aug 20, 2025
1 check passed
@aditya-css aditya-css deleted the feat/chat_list_pin_mute_icons branch August 20, 2025 08:14
dhyash-simform added a commit that referenced this pull request Sep 16, 2025
- Provided timestamp in last message time builder
- Provided chat data in profile avatar, chat & trailing builders
aditya-css pushed a commit that referenced this pull request Sep 22, 2025
- Provided timestamp in last message time builder
- Provided chat data in profile avatar, chat & trailing builders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants