Project

General

Profile

Actions

Feature #3058

closed

Show issue history using tabs

Added by Anonymous about 15 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Category:
Issues
Target version:
Start date:
2009-03-26
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

We have some issues where there are so many Status/priority/custom fields changes that you have to scroll heaps to find the useful messages that users have written.

It would be nice to be able to filter out things from the issue history. For example, next to the "History" text you could have the links "Show All" and "Show Messages". Selecting "Show Messages" would only show non-empty messages. It would also be great to remember the last selected option for the remaining session at a minimum (globally, probably not per issue to avoid overheads), or have a default in the user preferences.


Files

show-all-journals.png (60.4 KB) show-all-journals.png screenshot1 (the feature is disabled) Go MAEDA, 2015-09-23 09:08
show-notes-only.png (38.4 KB) show-notes-only.png screenshot2 (the feature is enabled) Go MAEDA, 2015-09-23 09:08
show-notes-only.diff (2.31 KB) show-notes-only.diff patch Go MAEDA, 2015-09-23 09:09
show-notes-only-2.diff (2.26 KB) show-notes-only-2.diff patch (updated) Go MAEDA, 2015-09-23 15:14
trac-show-property-changes.png (69.2 KB) trac-show-property-changes.png Go MAEDA, 2015-11-23 03:52
RedmineTabs.png (9.94 KB) RedmineTabs.png budo kaiman, 2015-11-23 13:32
JiraTabs.png (12.8 KB) JiraTabs.png budo kaiman, 2015-11-23 13:33
history-notes.png (52.1 KB) history-notes.png screenshot (notes; show-notes-only-v3) Go MAEDA, 2016-02-16 04:08
history-all.png (78.3 KB) history-all.png screenshot (history; show-notes-only-v3) Go MAEDA, 2016-02-16 04:08
show-notes-only-v3.diff (3.03 KB) show-notes-only-v3.diff patch (2016-02-16) Go MAEDA, 2016-02-16 05:18
issue_tabs.png (157 KB) issue_tabs.png Marius BĂLTEANU, 2017-05-01 16:29
01_allow-tabs-to-have-a-custom-action_v2_fixed.patch (1.45 KB) 01_allow-tabs-to-have-a-custom-action_v2_fixed.patch Andrey Lobanov (RedSoft), 2017-07-07 13:42
user_pref.png (207 KB) user_pref.png Marius BĂLTEANU, 2017-09-10 13:56
0001-allow-tabs-to-have-a-custom-action.patch (2.22 KB) 0001-allow-tabs-to-have-a-custom-action.patch Marius BĂLTEANU, 2018-12-16 15:59
0002-show-issue-history-in-tabs.patch (7.76 KB) 0002-show-issue-history-in-tabs.patch Marius BĂLTEANU, 2018-12-16 15:59
0004-add-time-entries-tab-to-issue-history-tabs.patch (5.2 KB) 0004-add-time-entries-tab-to-issue-history-tabs.patch Marius BĂLTEANU, 2018-12-16 15:59
0003-move-changesets-to-its-own-tab.patch (8.34 KB) 0003-move-changesets-to-its-own-tab.patch Marius BĂLTEANU, 2018-12-16 15:59
0006-load-remote-tab-only-once.patch (1.57 KB) 0006-load-remote-tab-only-once.patch Marius BĂLTEANU, 2018-12-16 15:59
0007-user-preference-for-issue-history-default-tab.patch (6.79 KB) 0007-user-preference-for-issue-history-default-tab.patch Marius BĂLTEANU, 2018-12-16 15:59
0005-load-changesets-and-time-entries-tabs-async.patch (11.9 KB) 0005-load-changesets-and-time-entries-tabs-async.patch Marius BĂLTEANU, 2018-12-16 15:59
0001-allow-tabs-to-have-a-custom-action.patch (2.22 KB) 0001-allow-tabs-to-have-a-custom-action.patch Marius BĂLTEANU, 2019-06-20 08:19
0002-show-issue-history-in-tabs.patch (7.81 KB) 0002-show-issue-history-in-tabs.patch Marius BĂLTEANU, 2019-06-20 08:19
0004-add-time-entries-tab-to-issue-history-tabs.patch (5.2 KB) 0004-add-time-entries-tab-to-issue-history-tabs.patch Marius BĂLTEANU, 2019-06-20 08:19
0003-move-changesets-to-its-own-tab.patch (8.5 KB) 0003-move-changesets-to-its-own-tab.patch Marius BĂLTEANU, 2019-06-20 08:19
0006-load-remote-tab-only-once.patch (1.57 KB) 0006-load-remote-tab-only-once.patch Marius BĂLTEANU, 2019-06-20 08:19
0005-load-changesets-and-time-entries-tabs-async.patch (11.9 KB) 0005-load-changesets-and-time-entries-tabs-async.patch Marius BĂLTEANU, 2019-06-20 08:19
0008-Remove-history-label-and-no-data.patch (1.28 KB) 0008-Remove-history-label-and-no-data.patch Marius BĂLTEANU, 2019-06-20 08:19
0007-user-preference-for-issue-history-default-tab.patch (6.35 KB) 0007-user-preference-for-issue-history-default-tab.patch Marius BĂLTEANU, 2019-06-20 08:19

Related issues

Related to Redmine - Feature #3046: Mix associated revisions with journal entriesNew2009-03-25

Actions
Related to Redmine - Feature #15409: Is it possible to view History section in pages?New

Actions
Related to Redmine - Feature #4487: Add better presentation of issue status historyNew2009-12-25

Actions
Related to Redmine - Feature #5061: Show time log entries in issue historyClosed2010-03-12

Actions
Related to Redmine - Feature #16052: Consider a better associated revisions block in the issue trackerClosed

Actions
Related to Redmine - Feature #21585: "Associated revisions" to be minimized / opened by clickClosed

Actions
Related to Redmine - Defect #31438: Incorrect position of the "Associated revisions" block when comments are displayed in reverse chronological orderClosedGo MAEDA

Actions
Related to Redmine - Defect #31599: 'selected' class disappears from the tab of the input field when switching the issue history tabClosedGo MAEDA

Actions
Related to Redmine - Defect #32198: Issues associated revisions not showing due to bad reference to lvar instead of ivarClosedGo MAEDA

Actions
Related to Redmine - Defect #32529: The end of the URL is replaced with "undefined" in IE11 and EdgeClosed

Actions
Related to Redmine - Defect #33338: Property changes tab does not show journals with both property changes and notesClosedGo MAEDA

Actions
Has duplicate Redmine - Feature #13298: Grouping issue history (activity) in issue history tabsClosed

Actions
Has duplicate Redmine - Feature #8536: Separate notes/comments from issue updatesClosed2011-06-06

Actions
Actions #1

Updated by Anonymous about 15 years ago

I didn't see any other issues like this when I submitted, but I just noticed that this is similar to #3037 - not a duplicate, but similar.

Actions #2

Updated by Anonymous about 15 years ago

I also just noticed that if #3046 were to be implemented, then it would work as another filter type in my proposal - "Show Associated Revisions"

Actions #3

Updated by Paul Quirk almost 15 years ago

I'm happy with Nick's proposal for filtering as this covers most of the problem I was talking about in #3037.

Actions #4

Updated by Paul Quirk almost 15 years ago

One other counter to this would be to set the "Show All" or "Show messages" flag at the role level. You could still have the toggle at the "history" level to override this.

Actions #5

Updated by Anonymous over 13 years ago

This issue seems to have grown stagnant but seems to be very helpful. At a minimum there should be the ability to filter out the history to show only the notes added to an issue (and vice versa).

Actions #6

Updated by Adam Sneller about 12 years ago

+1
There is an Issue History Tab plugin that works with 1.2.2. But the comment feature is buggy. Of course if anyone wants to take a shot a fixing it...

Here is a link to the issue:
https://github.com/tastapod/redmine_issue_history_tabs/issues/4

Actions #7

Updated by Kevin O over 11 years ago

+1. The way I would envision this is simply, for both default and custom fields, have an option to select whether or not they show in the Issue History.

For instance, we have a custom field called "Order" which we use for a custom backlog view to let product managers sort what's next in the pipeline. They move things around a lot, so the issue histories are cluttered up with perhaps a couple dozen or more messages of "Order changed to xx". Being able to select to hide changes to that field would be great.

Actions #8

Updated by Go MAEDA about 9 years ago

  • Related to Feature #15409: Is it possible to view History section in pages? added
Actions #9

Updated by Go MAEDA over 8 years ago

  • Related to Feature #8536: Separate notes/comments from issue updates added
Actions #10

Updated by Go MAEDA over 8 years ago

I wrote a patch to implement this issue.
This patch introduces a new checkbox "Show notes only". We can hide/show details by checking it.

showing all journals, like current Redmine:
screenshot1 (the feature is disabled)

showing notes only:
screenshot2 (the feature is enabled)

Actions #11

Updated by Go MAEDA over 8 years ago

Go MAEDA wrote:

I wrote a patch to implement this issue.
This patch introduces a new checkbox "Show notes only". We can hide/show details by checking it.

Updated the patch (code cleanup): show-notes-only-2.diff

Actions #12

Updated by Go MAEDA over 8 years ago

  • Target version set to Candidate for next major release
Actions #13

Updated by Go MAEDA over 8 years ago

Could this be included in 3.2.0? By this change, users can easily see all conversation on a issue. It increases productivity.

trac already has this feature (see the following screenshot).

Actions #14

Updated by budo kaiman over 8 years ago

I don't think a checkbox looks very good there. I'm using the Redmine Issue Detailed Tabs & Time plugin which adds tabs for different sections. This looks much better than a toggle check and offers more possible functionality. Tabs are used in Jira and it looks much cleaner.

Redmine plugin:

Jira:

Actions #15

Updated by Alexander Ryabinovskiy over 8 years ago

budo kaiman wrote:

I don't think a checkbox looks very good there. I'm using the Redmine Issue Detailed Tabs & Time plugin which adds tabs for different sections. This looks much better than a toggle check and offers more possible functionality. Tabs are used in Jira and it looks much cleaner.

I Agree, Jira`s tabs variant is much better. It can be extended to show "Work Log", "Commits' tabs in the future.

Actions #16

Updated by Sebastian Paluch over 8 years ago

+1 for tabs

Actions #17

Updated by Jean-Philippe Lang over 8 years ago

  • Target version changed from 3.2.0 to 3.3.0

I think we need a bit more time for this and 3.2.0 is now scheduled for the next week.

Actions #18

Updated by Alexander Lyzhenkov over 8 years ago

+1

Actions #19

Updated by Go MAEDA about 8 years ago

Here is a updated patch: show-notes-only-v3.diff
We can use tabs to switch views.

"All" tab (notes, details) :
screenshot (history; show-notes-only-v3)

"Notes" tab (notes only) :
screenshot (notes; show-notes-only-v3)

Actions #20

Updated by Go MAEDA about 8 years ago

Sorry, some files were missing in the patch.

Actions #21

Updated by Go MAEDA about 8 years ago

  • File deleted (show-notes-only-v3.diff)
Actions #22

Updated by Marius BĂLTEANU about 8 years ago

Hi,

The solution with 2 tabs is good, but in my opinion it'll be very helpful for the users to have also the following tabs available:
- Spent time: with all the spent times added to the respective issue (of course, depending on the user rights)
- Activity: all the entries without notes

That means 4 tabs in totals:
- All (activity + notes)
- Activity
- Notes
- Spent Times

Maybe also the associated revisions can be added as tab, but I'm not sure about this one :)

If you're agree with this proposal we can contribute with an updated patch which implement the tabs.
Go MAEDA wrote:

Here is a updated patch: show-notes-only-v3.diff
We can use tabs to switch views.

Actions #23

Updated by budo kaiman about 8 years ago

Marius Balteanu wrote:

Hi,

The solution with 2 tabs is good, but in my opinion it'll be very helpful for the users to have also the following tabs available:
- Spent time: with all the spent times added to the respective issue (of course, depending on the user rights)
- Activity: all the entries without notes

That means 4 tabs in totals:
- All (activity + notes)
- Activity
- Notes
- Spent Times

Spent time would be a good tab to have, but Activity should definitely be part of this.

Marius Balteanu wrote:

Maybe also the associated revisions can be added as tab, but I'm not sure about this one :)

Personally, I think that the associated revisions should be a tab and not have the weird little sidebar thing. If the patch to implement tabs is accepted, moving revisions to a tab should be done as a new feature, not part of this.


One general comment that I have about the patch (just from a quick glance, if I missed something let me know) is that the default tab probably shouldn't always be "All". This could probably be handled in a few ways, below are some options (the second being my preferred method):

  • Add a default tab setting
  • Remember the last tab the user used with "All" as default at first
Actions #24

Updated by Rupesh J almost 8 years ago

Hi,
During this implementation, Can you please also consider to provide a hook.
We have already defined some tabs in the ticket history region through a plugin development(currently overriding the corresponding partial).
Thanks.

Actions #25

Updated by Jean-Philippe Lang almost 8 years ago

  • Target version changed from 3.3.0 to 3.4.0

There's still some work to implement what is requested here (additional tabs, hook, memorization of last tab...)

Actions #26

Updated by Go MAEDA almost 8 years ago

Jean-Philippe Lang wrote:

There's still some work to implement what is requested here (additional tabs, hook, memorization of last tab...)

I think that we can implement only show-notes-only-v3.diff in 3.3.0 and other requests in future releases.

Actions #27

Updated by Dipan Mehta almost 8 years ago

This is great work going on.

There is another issue where the same was discussed in great detail - #4487 Do check and try to use some of these ideas.

Add related #4487.

Actions #28

Updated by Go MAEDA almost 8 years ago

  • Related to Feature #4487: Add better presentation of issue status history added
Actions #29

Updated by Marc D. over 7 years ago

Go MAEDA wrote:

Jean-Philippe Lang wrote:

There's still some work to implement what is requested here (additional tabs, hook, memorization of last tab...)

I think that we can implement only show-notes-only-v3.diff in 3.3.0 and other requests in future releases.

We got some issues with time spent feature after applying show-notes-only-v3.diff
If you click on the spent time summit in the issues description ( which links to /issues/1111/time_entries ) the issues just reloads instead of showing a list of all time entries for the issue.
Anyone else got this problem?

Actions #30

Updated by Artem - over 7 years ago

Mark Mulder, did you manage to resolve the issue with time entries?

Actions #31

Updated by Jean-Philippe Lang about 7 years ago

  • Target version changed from 3.4.0 to Candidate for next major release

Still some work to be done.

Actions #32

Updated by Marius BĂLTEANU almost 7 years ago

  • File 01_allow-tabs-to-have-a-custom-action.patch added
  • File 02_show-issue-history-in-tabs.patch added
  • File 03_move-changesets-to-its-own-tab.patch added
  • File 04_add-time-entries-tab-to-issue-history-tabs.patch added
  • File 05_load-changesets-and-time-entries-tabs-async.patch added
  • File 06_load-remote-tab-only-once.patch added
  • File issue_tabs.png issue_tabs.png added

I've attached a series of patches that implement this feature. Because the change is quite big, I've split the changes in multiple patches.

01_allow-tabs-to-have-a-custom-action.patch

This patch allows render_tabs to have custom action for onclick and also, make the partial rendering optional. In this way, tabs can use the same div (partial) for displaying content.

02_show-issue-history-in-tabs.patch

This patch display issue history in three tabs:
  • All: all journals
  • Notes: only journals with notes
  • History: only journals with history changes and without notes

The notes tab is default when the tab parameters doesn't exists.

03_move-changesets-to-its-own-tab.patch

This patch moves the associated revisions to a new tab and uses the look and feel from the journals to display the changesets.

04_add-time-entries-tab-to-issue-history-tabs.patch

This patch adds a new tab with the issue's visible spent times. The same look and feel from journals is used. Related to #5061.

05_load-changesets-and-time-entries-tabs-async.patch

This patch changes the way in which the changesets and time entries tabs are loaded in order to use ajax calls only on user request. In this way, we speed up a little bit the issue show page. Journals are still loaded in the same time with the issue.

06_load-remote-tab-only-once.patch

I think it doesn't worth it to make an ajax call each time when the user clicks on changeset or time entries tabs

Let's discuss this implementation for Redmine version:3.5.0 (I think that the change is too big for 3.4.0). In Zitec, we're using from the beginning some plugins that implement the tabs, but each time we had multiple issues with them. I really think that this feature is needed and also, having the tabs async implemented, we can add more features in the future (some examples: affected branches and files from git revisions, time in state..).

Regarding the memorization of last tab, I'm not sure that is a good idea because if I'm on the changeset tab of the current issue, it doesn't mean that I want to see the same tab first when I open the next issue. If it's really required, I prefer to have it configurable.

Any feedback is appreciated (attached is a printscreen with all 5 tabs).

Actions #33

Updated by Go MAEDA almost 7 years ago

  • Target version changed from Candidate for next major release to 4.1.0

Jean-Philippe Lang wrote:

Still some work to be done.

New patches have been submitted by Marius BALTEANU.
Let's consider implementing this feature.

Actions #34

Updated by budo kaiman almost 7 years ago

Any feedback is appreciated (attached is a printscreen with all 5 tabs).

This looks excellent, one small comment I have is that it would be nice to have avatars shown in the "Associated Revisions" tab like they are for other journals.

Regarding the memorization of last tab, I'm not sure that is a good idea because if I'm on the changeset tab of the current issue, it doesn't mean that I want to see the same tab first when I open the next issue. If it's really required, I prefer to have it configurable.

I can understand why you feel that way, but I still believe it would be useful. Particularly, if I wanted to go through the time log or commit history of a bunch of issues, I would scroll through and open them. If the last tab was remembered then I'd only have to change tab once when I started and then change it back to whatever I want to see when I open the next issue after I'm done. If it were a configurable setting then I'd either have to change the tab for each issue I view, or navigate the settings to change the default before and then after, both of which can be a bit cumbersome. If you still feel opposed to it, that's fine but I'd at least like to express why I feel it would be valuable.

Actions #35

Updated by Marius BĂLTEANU almost 7 years ago

budo kaiman wrote:

This looks excellent, one small comment I have is that it would be nice to have avatars shown in the "Associated Revisions" tab like they are for other journals.

The avatars are already added, but they are missing from the screenshot because the feature is disabled on my local environment.

I can understand why you feel that way, but I still believe it would be useful. Particularly, if I wanted to go through the time log or commit history of a bunch of issues, I would scroll through and open them. If the last tab was remembered then I'd only have to change tab once when I started and then change it back to whatever I want to see when I open the next issue after I'm done. If it were a configurable setting then I'd either have to change the tab for each issue I view, or navigate the settings to change the default before and then after, both of which can be a bit cumbersome. If you still feel opposed to it, that's fine but I'd at least like to express why I feel it would be valuable.

It is a valid point of view. What about having the "Last visited tab" as an option in the default tab setting (along with the tabs)? Basically, each Redmine administrator can set as default tab setting for issues one of the following options: All, Notes, History, Spent time, Associated Revisions and Last visited tab.

Actions #36

Updated by budo kaiman almost 7 years ago

It is a valid point of view. What about having the "Last visited tab" as an option in the default tab setting (along with the tabs)? Basically, each Redmine administrator can set as default tab setting for issues one of the following options: All, Notes, History, Spent time, Associated Revisions and Last visited tab.

I think that is a very sensible solution, though I'm not sure if it would be better as a user preference than an administrator setting.

Actions #37

Updated by Mischa The Evil almost 7 years ago

  • Related to Feature #12194: Source control revisions should be integrated as part of the history added
Actions #38

Updated by Toshi MARUYAMA almost 7 years ago

  • Related to Feature #5061: Show time log entries in issue history added
Actions #39

Updated by Mischa The Evil almost 7 years ago

I really do like the tabbed solution for this issue.

My two cents on the patches provided in note-32 based on a visual scan of the patches and the posted screenshot:
  • 'All' tab does not include spent time entries and associated revisions (this seems logical seeing where we are coming from, but it might be unexpected behaviour for an 'All' tab)
    • and if these were to be merged into the 'All' tab too, then it would provide a false overview of issues' journals as their items aren't journals (only notes and property changes are journals)
  • to see all the associated revisions of an issue, one now has to click one more time (on the tab) than before (they were immediately visible along the history), which might be a drawback for some (I can live with it though)
  • I think that a 'properties' or 'property changes' label for the 'history' tab is better considering its content and the terminology used throughout Redmine (eg. see the label of the issue edit form [which is 'change properties'])
  • additional plugin hooks for the spent time and associated revisions tab views are not included
  • regarding the last visited tab memoization: I like the solution proposed by Marius in note-35, but, as Budo, I also like to see such a setting as a user preference instead of a system-wide (admin) setting

All in all I'm pretty positive about the patch series. This feature seems to be coming along nicely... Thanks Marius for your work on this.

Actions #40

Updated by Marius BĂLTEANU almost 7 years ago

Thanks Mischa for your feedback.

Mischa The Evil wrote:

I really do like the tabbed solution for this issue.
My two cents on the patches provided in note-32 based on a visual scan of the patches and the posted screenshot:
  • 'All' tab does not include spent time entries and associated revisions (this seems logical seeing where we are coming from, but it might be unexpected behaviour for an 'All' tab)
    • and if these were to be merged into the 'All' tab too, then it would provide a false overview of issues' journals as their items aren't journals (only notes and property changes are journals)

1. It is better to rename the 'All' tab with the current label 'History'?
2. It'll be nice to have the time entries and associated revisions merged in an 'All' or 'Activity' tab, but maybe it'll be done in a future version.

  • to see all the associated revisions of an issue, one now has to click one more time (on the tab) than before (they were immediately visible along the history), which might be a drawback for some (I can live with it though)
  • I think that a 'properties' or 'property changes' label for the 'history' tab is better considering its content and the terminology used throughout Redmine (eg. see the label of the issue edit form [which is 'change properties'])

3. 'Changed properties' sounds good for you?

  • additional plugin hooks for the spent time and associated revisions tab views are not included

4. I'll a update the patches to include the hooks.

  • regarding the last visited tab memoization: I like the solution proposed by Marius in note-35, but, as Budo, I also like to see such a setting as a user preference instead of a system-wide (admin) setting

5. I'll create a new patch for this feature. I think the best way to implement this is to have the both settings: one in admin to configure the default value for new users, and one as a user preference.

Actions #41

Updated by Marius BĂLTEANU over 6 years ago

  • File 01_allow-tabs-to-have-a-custom-action_v2.patch added
  • File 02_show-issue-history-in-tabs_v2.patch added
  • File 03_move-changesets-to-its-own-tab_v2.patch added
  • File 04_add-time-entries-tab-to-issue-history-tabs_v2.patch added
  • File 05_load-changesets-and-time-entries-tabs-async_v2.patch added
Updated the patch series (01-05) in order to apply cleanly to the current trunk and to include the feedback from Mischa:
  • Renamed the tab "All" to "History"
  • Renamed the tab "History" to "Property changes"
  • Added hooks for changeset and time entries tabs
  • Updated tests

Please remove the first version of patches (except 06_load-remote-tab-only-once.patch)

Actions #42

Updated by Mischa The Evil over 6 years ago

  • File deleted (01_allow-tabs-to-have-a-custom-action.patch)
Actions #43

Updated by Mischa The Evil over 6 years ago

  • File deleted (02_show-issue-history-in-tabs.patch)
Actions #44

Updated by Mischa The Evil over 6 years ago

  • File deleted (03_move-changesets-to-its-own-tab.patch)
Actions #45

Updated by Mischa The Evil over 6 years ago

  • File deleted (04_add-time-entries-tab-to-issue-history-tabs.patch)
Actions #46

Updated by Mischa The Evil over 6 years ago

  • File deleted (05_load-changesets-and-time-entries-tabs-async.patch)
Actions #47

Updated by Mischa The Evil over 6 years ago

  • Subject changed from User Filtering of Issue History to Show issue history using tabs

Marius BALTEANU wrote:

Updated the patch series (01-05) [...]

Nice! Thanks for incorporating my feedback. To summarize the remaining outstanding tasks:
  • Add setting(s) for memoization of the last visited tab
    • I don't think we need an admin setting as well as a user preference though. I think that only having a user preference will suffice.
  • Merge time entries and associated revisions with journals into an 'All' or 'Activity' tab

Please remove the first version of patches (except 06_load-remote-tab-only-once.patch)

Done.

Actions #48

Updated by Marius BĂLTEANU over 6 years ago

Thanks!

Mischa The Evil wrote:

  • Add setting(s) for memoization of the last visited tab
    • I don't think we need an admin setting as well as a user preference though. I think that only having a user preference will suffice.

Is on my to do list.

  • Merge time entries and associated revisions with journals into an 'All' or 'Activity' tab

I'll work on this after these patches are committed because the changes are quite big, hard to maintain and also, to be honest, I need a confirmation from Jean-Philippe Lang that we're on the right way with the implementation.

Actions #49

Updated by Mischa The Evil over 6 years ago

Marius BALTEANU wrote:

  • Merge time entries and associated revisions with journals into an 'All' or 'Activity' tab

I'll work on this after these patches are committed because the changes are quite big, hard to maintain and also, to be honest, I need a confirmation from Jean-Philippe Lang that we're on the right way with the implementation.

Please don't get me wrong. I didn't mean to say that this particular change is a requirement before this patch serial can make it into the core. It is something that could well be handled through a separate issue after integration of this issue. I just wanted to give a summary of things mentioned throughout this issue which are not (yet) covered by your patches.

Actions #50

Updated by Wim DePreter over 6 years ago

If you change presentation of associated revisions, maybe you could keep #13715 in mind?

Actions #51

Updated by Andrey Lobanov (RedSoft) over 6 years ago

Here is a fixed version of the patch. While viewing the repository entries, there was some problem - Tabs didn't opening, because of 'onclick' action.

Actions #52

Updated by Marius BĂLTEANU over 6 years ago

  • File user_pref.png user_pref.png added
  • File 01_allow-tabs-to-have-a-custom-action_r16963.patch added
  • File 02_show-issue-history-in-tabs_r16963.patch added
  • File 03_move-changesets-to-its-own-tab_r16963.patch added
  • File 04_add-time-entries-tab-to-issue-history-tabs_r16963.patch added
  • File 05_load-changesets-and-time-entries-tabs-async_r16963.patch added
  • File 06_load-remote-tab-only-once_r16963.patch added
  • File 07_user_preference_for_issue_history_default_tab_r16963.patch added
  • Assignee set to Jean-Philippe Lang

Andrey Lobanov, thanks for finding and fixing the issue. I reworked your fix a little bit in order to clean up the view file.

I'm attaching the updated patches which cleanly apply on the current trunk (r16963) and incorporate the changes from #26638 and test fixes for rails 5.

Also, I've finished the last patch for this ticket (07_user_preference_for_issue_history_default_tab_r16963.patch) which adds the possibility for the users to select the default issue history tab as preference. The following options are available: notes (default), history, properties, time_entries, changesets and last visited tab.

Jean-Philippe Lang, I think that I've covered all the required changes and the patches are ready for review. I'm assigning you in order to take a call regarding them (it'll be very useful for me if you can do this as soon as you can because there are a lot of changes and it is hard and time consuming for me to maintain these 7 patches).

Actions #53

Updated by budo kaiman over 6 years ago

This looks great, thanks for all your hard work on this. Hopefully it can be incorporated soon.

Actions #54

Updated by Go MAEDA over 6 years ago

  • Related to Feature #16052: Consider a better associated revisions block in the issue tracker added
Actions #55

Updated by Mischa The Evil over 6 years ago

  • Related to deleted (Feature #12194: Source control revisions should be integrated as part of the history)
Actions #56

Updated by Go MAEDA about 6 years ago

  • Related to Feature #21585: "Associated revisions" to be minimized / opened by click added
Actions #57

Updated by Go MAEDA almost 6 years ago

  • Has duplicate Feature #13298: Grouping issue history (activity) in issue history tabs added
Actions #58

Updated by Marius BĂLTEANU over 5 years ago

  • Target version changed from 4.1.0 to 4.0.0

I'm setting target version to 4.0.0 in order to get at least a feedback from Jean-Philippe Lang. It is a long awaited feature, with more than 30 watchers, a lot of related/duplicated issues and patches dragged from version 3.2.0. Also, I'm sure that once we implement this feature, we can make a lot of improvements on top of it.

Actions #59

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (01_allow-tabs-to-have-a-custom-action_v2.patch)
Actions #60

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (02_show-issue-history-in-tabs_v2.patch)
Actions #61

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (06_load-remote-tab-only-once.patch)
Actions #62

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (04_add-time-entries-tab-to-issue-history-tabs_v2.patch)
Actions #63

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (03_move-changesets-to-its-own-tab_v2.patch)
Actions #64

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (05_load-changesets-and-time-entries-tabs-async_v2.patch)
Actions #65

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (01_allow-tabs-to-have-a-custom-action_r16963.patch)
Actions #66

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (05_load-changesets-and-time-entries-tabs-async_r16963.patch)
Actions #67

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (07_user_preference_for_issue_history_default_tab_r16963.patch)
Actions #68

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (06_load-remote-tab-only-once_r16963.patch)
Actions #69

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (04_add-time-entries-tab-to-issue-history-tabs_r16963.patch)
Actions #70

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (03_move-changesets-to-its-own-tab_r16963.patch)
Actions #71

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (02_show-issue-history-in-tabs_r16963.patch)
Actions #72

Updated by Marius BĂLTEANU over 5 years ago

  • File 0001-allow-tabs-to-have-a-custom-action.patch added
  • File 0002-show-issue-history-in-tabs.patch added
  • File 0003-move-changesets-to-its-own-tab.patch added
  • File 0004-add-time-entries-tab-to-issue-history-tabs.patch added
  • File 0005-load-changesets-and-time-entries-tabs-async.patch added
  • File 0006-load-remote-tab-only-once.patch added
  • File 0007-user-preference-for-issue-history-default-tab.patch added

Updated the patch series to apply cleanly on the current trunk version.

Actions #73

Updated by Jean-Philippe Lang over 5 years ago

  • Target version deleted (4.0.0)

Thank you for the patches series. I hope I will be able to review it soon but I have to unassign this from 4.0 if we want to release before the end of september.

Actions #74

Updated by Go MAEDA over 5 years ago

  • Target version set to 4.1.0

I have to unassign this from 4.0 if we want to release before the end of september.

It really makes sense to prioritize things to release 4.0 as soon as possible. Let's consider to deliver this in 4.1.0 :)

Actions #75

Updated by Marius BĂLTEANU over 5 years ago

Go MAEDA wrote:

I have to unassign this from 4.0 if we want to release before the end of september.

It really makes sense to prioritize things to release 4.0 as soon as possible. Let's consider to deliver this in 4.1.0 :)

I’m happy with this decision.

Actions #76

Updated by Bernhard Rohloff over 5 years ago

As it's the end of November now and the release hasn't happened yet, may we reconsider an integration of this feature in 4.0.0 again?
The issue is quite old and provides a high UX improvement which I think many folks would love to see in the next major release.

Actions #77

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0001-allow-tabs-to-have-a-custom-action.patch)
Actions #78

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0002-show-issue-history-in-tabs.patch)
Actions #79

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0003-move-changesets-to-its-own-tab.patch)
Actions #80

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0004-add-time-entries-tab-to-issue-history-tabs.patch)
Actions #81

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0005-load-changesets-and-time-entries-tabs-async.patch)
Actions #82

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0006-load-remote-tab-only-once.patch)
Actions #83

Updated by Marius BĂLTEANU over 5 years ago

  • File deleted (0007-user-preference-for-issue-history-default-tab.patch)
Actions #85

Updated by Go MAEDA almost 5 years ago

  • Related to Defect #31438: Incorrect position of the "Associated revisions" block when comments are displayed in reverse chronological order added
Actions #86

Updated by Ko Nagase almost 5 years ago

Hi,
I hope that this functionality will be included on next Redmine 4.1.0 release.
Is there anything that I can help with this ?

Actions #88

Updated by Jean-Philippe Lang almost 5 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Thanks Marius for updating the patches. They are now committed, thanks for your hard work on this!

Actions #89

Updated by Marius BĂLTEANU almost 5 years ago

  • Related to Defect #31599: 'selected' class disappears from the tab of the input field when switching the issue history tab added
Actions #90

Updated by Marius BĂLTEANU almost 5 years ago

  • Status changed from Closed to Reopened

A small problem was reported by Mizuki in #31599. I've attached there the fix.

Actions #91

Updated by Marius BĂLTEANU almost 5 years ago

  • Related to deleted (Feature #8536: Separate notes/comments from issue updates)
Actions #92

Updated by Marius BĂLTEANU almost 5 years ago

  • Has duplicate Feature #8536: Separate notes/comments from issue updates added
Actions #93

Updated by Go MAEDA almost 5 years ago

  • Status changed from Reopened to Closed

Marius BALTEANU wrote:

A small problem was reported by Mizuki in #31599. I've attached there the fix.

Committed the patch in r18311. Thanks.

Actions #95

Updated by Marius BĂLTEANU over 4 years ago

  • Status changed from Closed to Reopened
  • Assignee changed from Jean-Philippe Lang to Marius BĂLTEANU
Actions #96

Updated by Yuichi HARADA over 4 years ago

This patch works with IE11.

diff --git a/public/javascripts/application.js b/public/javascripts/application.js
index 7a2805d97..019b45dc4 100644
--- a/public/javascripts/application.js
+++ b/public/javascripts/application.js
@@ -385,7 +385,8 @@ function showIssueHistory(journal, url) {
   return false;
 }

-function getRemoteTab(name, remote_url, url, load_always = false) {
+function getRemoteTab(name, remote_url, url, load_always) {
+  load_always = load_always || false;
   var tab_content = $('#tab-content-' + name);

   tab_content.parent().find('.tab-content').hide();

This also solves.
https://www.redmine.org/issues/31433#note-12

Actions #97

Updated by Marius BĂLTEANU over 4 years ago

  • Assignee deleted (Marius BĂLTEANU)

Yuichi HARADA wrote:

This patch works with IE11.

[...]

This also solves.
https://www.redmine.org/issues/31433#note-12

Thanks Yuichi for catching this issue. Your fix is good, it can be committed.

Actions #98

Updated by Go MAEDA over 4 years ago

  • Status changed from Reopened to Closed

Yuichi HARADA wrote:

This patch works with IE11.

[...]

This also solves.
https://www.redmine.org/issues/31433#note-12

Committed. Thanks.

Actions #99

Updated by Go MAEDA over 4 years ago

  • Assignee set to Jean-Philippe Lang
Actions #100

Updated by Mischa The Evil over 4 years ago

  • Related to Defect #32198: Issues associated revisions not showing due to bad reference to lvar instead of ivar added
Actions #101

Updated by Go MAEDA over 4 years ago

  • Related to Defect #32529: The end of the URL is replaced with "undefined" in IE11 and Edge added
Actions #102

Updated by Go MAEDA over 4 years ago

  • Status changed from Closed to Reopened

A cross-Browser compatibility issue caused by the changes has been reported as #32529.

Actions #103

Updated by Marius BĂLTEANU over 4 years ago

Go MAEDA wrote:

A cross-Browser compatibility issue caused by the changes has been reported as #32529.

I've posted a fix which is good enough for now, but I'm still looking for a better solution for the future. The problem is caused by this line source:trunk/app/views/common/_tabs.html.erb#L27 which calls the onclick functions in a non element context.

Actions #104

Updated by Jean-Philippe Lang over 4 years ago

  • Status changed from Reopened to Closed
Actions #105

Updated by Ali Hamdan almost 4 years ago

Dears,
just an additional feature to mention it will be great if the PDF export of the issue will include the time entry(time+comment+custome fields) under history
Thanks.

Actions #106

Updated by Marius BĂLTEANU almost 4 years ago

Ali Hamdan wrote:

Dears,
just an additional feature to mention it will be great if the PDF export of the issue will include the time entry(time+comment+custome fields) under history
Thanks.

Please open a new issue for this request.

Actions #107

Updated by Go MAEDA almost 4 years ago

  • Related to Defect #33338: Property changes tab does not show journals with both property changes and notes added
Actions

Also available in: Atom PDF