Skip to content
Snippets Groups Projects

#1611 internal comments by default in tickets and visual cue to differentiate created comments

Merged #1611 internal comments by default in tickets and visual cue to differentiate created comments

Merge request reports

Approval is optional

Merged by avatar (Mar 29, 2025 5:20pm UTC)

Merge details

  • Changes merged into master with c05be24d.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
30 30 {if in_array($MODULE_NAME, $PRIVATE_COMMENT_MODULES)}
31 31 <div class="" style="margin: 7px 0;">
32 32 <label>
33 <input type="checkbox" id="is_private" style="margin:2px 0px -2px 0px">&nbsp;&nbsp;{vtranslate('LBL_INTERNAL_COMMENT')}
33 <input type="checkbox" id="is_private" style="margin:2px 0px -2px 0px" checked>&nbsp;&nbsp;{vtranslate('LBL_INTERNAL_COMMENT')}
34 34 </label>&nbsp;&nbsp;
  • @ruben.estrada I don't see the style change for internal comment background color, Can you please verify.

  • 30 30 {if in_array($MODULE_NAME, $PRIVATE_COMMENT_MODULES)}
    31 31 <div class="" style="margin: 7px 0;">
    32 32 <label>
    33 <input type="checkbox" id="is_private" style="margin:2px 0px -2px 0px">&nbsp;&nbsp;{vtranslate('LBL_INTERNAL_COMMENT')}
    33 <input type="checkbox" id="is_private" style="margin:2px 0px -2px 0px" checked>&nbsp;&nbsp;{vtranslate('LBL_INTERNAL_COMMENT')}
    34 34 </label>&nbsp;&nbsp;
    • yes @uma.s , if you look at line 30, you'll see that line 33 is inside an if statement that checks for that. But I'm not sure if you meant that. If no, could you please clarify a bit more?

  • @uma.s if you take look at the image I posted, you'll see that "internal comments" (the ones where I'm adding the closed lock icon) have a slightly yellow background. If you don't see it, that reinforces my argument about it not being obvious enough and leading to confusion.

    Here is the exact line where the background is changed: https://code.vtiger.com/ruben.estrada/vtigercrm/blob/7.4.0/layouts/v7/modules/Vtiger/RecentComments.tpl#L74

  • @ruben.estrada Ok, that looks fine.

    Thanks! for the clarification.

  • Uma mentioned in commit 3e7816b7

    mentioned in commit 3e7816b7

  • Uma Status changed to merged

    Status changed to merged

  • Please register or sign in to reply
    Loading