-
Notifications
You must be signed in to change notification settings - Fork 97
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
Subject line should show topic from newest message - eMBee's idea 3 #384
base: develop
Are you sure you want to change the base?
Conversation
i love this one, when it's working. some issues: to fix: figure out how sup decides which message to open and find out how to get the subject of that message. |
def find_last | ||
ret = nil | ||
find_all { |e| ret = yield(e) } | ||
ret || nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nil or nil?
Not sure if I fully support this idea, what if someone picks up an old thread from a mailinglist, modifies the subject and then sends a reply... also, is not the original subject best? Otherwise it might be more difficult to find an old thread and keep track of which is which (for the user). |
it is debatable. the problem for me is, if i read a thread, and i found i don't care about it, but then the subject changes, i want to know about it |
You will now about it because there is a new message, thread splitting on subjects does not make so much sense. That would change the way the threading in the index is built up. There is a hook for the date widget, maybe making a hook for the subject widget would be a better option? |
just having a new message doesn't tell me enough. the problem is, that i don't know if the message is the same old subject i am not interested in, or if the subject has changed. do you mean a hook to make the subject configurable? that could be interesting... |
Excerpts from eMBee's message of January 18, 2015 15:57:
Yes, please have a look at the date-widget and size-widget hook. |
https://factor.cc/pad/p/sup-ideas