Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-10779

KRAD Demo Library Rich Messages CSS demo not showing changed presentation

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.4
    • Fix Version/s: 2.4
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KRAD
    • Sprint:
      2.4.0-rc1 Sprint 4
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      KRAD Demo Library Rich Messages CSS sets fl-text-underline fl-text-larger but demonstation text does not present as changed.

        Attachments

          Issue Links

            Activity

            cpedersen Corey Pedersen (Inactive) created issue -
            cpedersen Corey Pedersen (Inactive) made changes -
            Field Original Value New Value
            Link This issue discovered by KULRICE-10552 [ KULRICE-10552 ]
            cpedersen Corey Pedersen (Inactive) made changes -
            Link This issue is related to KULRICE-9875 [ KULRICE-9875 ]
            Hide
            cpedersen Corey Pedersen (Inactive) added a comment -

            fl-text-underline fl-text-larger are Fluid CSS tags.

            Show
            cpedersen Corey Pedersen (Inactive) added a comment - fl-text-underline fl-text-larger are Fluid CSS tags.
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.3.3 [ 17113 ]
            thrclark Tom Clark made changes -
            Assignee Tom Clark [ thrclark ]
            Hide
            bsmith Brian Smith (Inactive) added a comment -

            just add new classes to replace these and set those instead

            Show
            bsmith Brian Smith (Inactive) added a comment - just add new classes to replace these and set those instead
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.4 [ 16913 ]
            Fix Version/s 2.3.3 [ 17113 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s Backlog [ 15811 ]
            Fix Version/s 2.4 [ 16913 ]
            shahess Shannon Hess made changes -
            Assignee Tom Clark [ thrclark ] Shannon Hess [ shahess ]
            shahess Shannon Hess made changes -
            Link This issue is duplicated by KULRICE-11922 [ KULRICE-11922 ]
            shahess Shannon Hess made changes -
            Affects Version/s 2.4 [ 16913 ]
            shahess Shannon Hess made changes -
            Fix Version/s 2.4 [ 16913 ]
            Fix Version/s Backlog [ 15811 ]
            shahess Shannon Hess made changes -
            Sprint 2.4.0-rc1 Sprint 4 [ 231 ]
            shahess Shannon Hess made changes -
            Rank Ranked lower
            Hide
            shahess Shannon Hess added a comment -

            I created new css classes to replace the fluid classes that were removed, and changed references to fl-text-underline and fl-text-larger so that the new css classes are used.

            Show
            shahess Shannon Hess added a comment - I created new css classes to replace the fluid classes that were removed, and changed references to fl-text-underline and fl-text-larger so that the new css classes are used.
            shahess Shannon Hess made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            shahess Shannon Hess logged work - 18/Mar/14 12:08 AM
            • Time Spent:
              2 hours
               
              Made needed changes and tested
            shahess Shannon Hess made changes -
            Remaining Estimate 0 minutes [ 0 ]
            Time Spent 2 hours [ 7200 ]
            Worklog Id 93864 [ 93864 ]
            Hide
            shahess Shannon Hess added a comment -

            setting estimate to match what was set for KULRICE-11922

            Show
            shahess Shannon Hess added a comment - setting estimate to match what was set for KULRICE-11922
            shahess Shannon Hess made changes -
            Original Estimate 3 hours [ 10800 ]
            Hide
            bsmith Brian Smith (Inactive) added a comment - - edited

            Sorry my comment was old on this one. There wasn't any bootstrap equivalent classes?

            Show
            bsmith Brian Smith (Inactive) added a comment - - edited Sorry my comment was old on this one. There wasn't any bootstrap equivalent classes?
            Hide
            shahess Shannon Hess added a comment -

            I couldn't find ones that exactly matched the fluid classes. There was a lot of css classes that had used the underline, but the names did not make sense - such as hover or link. To replace fl-text-larger, I could probably use uif-applicationMessage since it only adds one other thing in addition to the size change. Would you recommend that?

            Existing class I could use for fl-text-larger:

            .uif-applicationMessage {
              font-size: 125%;
              font-weight: bold;
            }
            

            Classes I added:

            .uif-text-underline {
              text-decoration: underline;
            }
            
            .uif-text-larger {
              font-size: 125%;
            }
            
            Show
            shahess Shannon Hess added a comment - I couldn't find ones that exactly matched the fluid classes. There was a lot of css classes that had used the underline, but the names did not make sense - such as hover or link. To replace fl-text-larger, I could probably use uif-applicationMessage since it only adds one other thing in addition to the size change. Would you recommend that? Existing class I could use for fl-text-larger: .uif-applicationMessage { font-size: 125%; font-weight: bold; } Classes I added: .uif-text-underline { text-decoration: underline; } .uif-text-larger { font-size: 125%; }
            Hide
            bsmith Brian Smith (Inactive) added a comment -

            Its fine what you did then, thanks for looking

            Show
            bsmith Brian Smith (Inactive) added a comment - Its fine what you did then, thanks for looking
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

              People

              • Assignee:
                shahess Shannon Hess
                Reporter:
                cpedersen Corey Pedersen (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 3 hours
                  3h
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 hours Time Not Required
                  2h