VOOZH about

URL: https://bugzilla.mozilla.org/show_bug.cgi?id=1967073

⇱ 1967073 - Remove unused motifSettings code from cZ


Closed Bug 1967073 Opened 1 year ago Closed 10 months ago

Remove unused motifSettings code from cZ

Remove unused motifSettings code from cZ
SeaMonkey
Chat
Trunk
All
All
RESOLVED FIXED
RESOLVED
FIXED
Future
Tracking Status
seamonkey2.53 + fixed
Tracking Status
seamonkey2.53
seamonkey2.57esr
---
SM2.53.22
QA Whiteboard:
---
Change Request:
---
Bug Flags:
Signature:
None
This bug is publicly visible.

 
Assignee

Description

1 year ago

As default information of the same type that is not an action is collapsed are and there's a pref to control collapsing actions of the same type. The motifSettings code is used for motifs that want to collapse everything down for consecutive information from a particular user regardless of the type of information. None of the motifs do this, so let's removed this code and simplify / tidy things around it.

Assignee

Comment 1

1 year ago

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.22pre
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9488462 - Flags: review?(frgrahl)
Attachment #9488462 - Flags: approval-comm-release?
Whiteboard: SM2.53.22

Comment on attachment 9488462 [details] [diff] [review]
1967073-irc-remove-motifsettings-25322.patch

LGTM r/a+

Attachment #9488462 - Flags: review?(frgrahl)
Attachment #9488462 - Flags: review+
Attachment #9488462 - Flags: approval-comm-release?
Attachment #9488462 - Flags: approval-comm-release+

Comment 3

10 months ago

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/aac760e19c9c
Remove unused motifSettings code from cZ. r=frg DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.