Skip to content
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

RevGrid multiline indicators 3.5 #9253

Merged

Conversation

gerhardol
Copy link
Member

@gerhardol gerhardol commented Jun 1, 2021

Two issues from #9243 for 3.5
(unknown when the regressions occurred)

Proposed changes

These issues relate to that the commit body is not stored for commits older than 6 months.
The subject and info if the commit has multiline info is always available.
If you select the commit in the grid, the full body is loaded (again).

  • RevGrid tooltip: Only show "click to load" for multiline
    This occurred if the commit had any related refs.

  • Multiline indicator only shown if Body is available
    Note that there is no tooltip if there are no refs or multiline. Without the multiline indicator it is hard to understand why.

Screenshots

before after
image image

Test methodology

manual


✒️ I contribute this code under The Developer Certificate of Origin.

@ghost ghost assigned gerhardol Jun 1, 2021
@gerhardol gerhardol changed the title Feature/multiline indicators 3.5 RevGrid multiline indicators 3.5 Jun 1, 2021
@gerhardol gerhardol changed the base branch from master to release/3.5 June 1, 2021 22:09
Copy link
Member

@mstv mstv left a comment

Choose a reason for hiding this comment

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

👍 Works well

@mstv mstv closed this Jun 3, 2021
@mstv mstv reopened this Jun 3, 2021
@RussKie RussKie merged commit c8766e1 into gitextensions:release/3.5 Jun 5, 2021
@RussKie RussKie added this to the 3.5.1 milestone Jun 5, 2021
@gerhardol gerhardol deleted the feature/multiline-indicators-3.5 branch June 5, 2021 16:45
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 18, 2021
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 18, 2021
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 18, 2021
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 18, 2021
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 20, 2021
gerhardol added a commit to gerhardol/gitextensions that referenced this pull request Jun 20, 2021
gerhardol added a commit that referenced this pull request Jun 20, 2021
Cherrypicked with review comments from d7be031 (#9253)
gerhardol added a commit that referenced this pull request Jun 20, 2021
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.

None yet

3 participants