Details

    • Business Value:
      5
    • Similar Issues:
      Show 5 results 

      Description

      Hi there,
      I mentioned this to Aaron previously, but wanted to capture it in a ticket.

      When a user goes to their user notifications, and they click Confirm or Ignore, it should not require an entire page refresh. What we can do is use javascript to make the request to the link, and add a wait cursor until it's finished, and then remove the row once it's done.

      I have some code that I hacked together that will get this to work:

      AUI().use('aui-base', 'aui-io-request', function(A) {
      	A.one('#_7_WAR_soportlet_notificationsMenuContent').delegate(
      		'click',
      		function(event) {
      			event.preventDefault();
      
      			var row = event.currentTarget.ancestor('.user-notification-event-content');
      			var loadingRow = A.Node.create('<div class="loading-animation"></div>');
      
      			row.hide().placeAfter(loadingRow);
      
      			A.io.request(
      				event.currentTarget.attr('href'), 
      				{
      					on: {
      						success: function() {
      							row.remove();
      							loadingRow.remove();
      						}
      					}
      				}
      			);
      		},
      		'a'
      	);
      });
      

        Issue Links

          Activity

          Hide
          Vicki Tsang added a comment -

          Thanks Nate! I will forward this on to Aaron to implement.

          Show
          Vicki Tsang added a comment - Thanks Nate! I will forward this on to Aaron to implement.
          Hide
          mandy.zia added a comment -

          Committed at:

          Plugins 6.1.x CE GIT ID: 77113c13de4ee973be8fe2df360f94e35356e6db.
          Plugins 6.1.x EE GIT ID: 2fbac957efca18f78f7db5354a61892098b4cb30.

          Show
          mandy.zia added a comment - Committed at: Plugins 6.1.x CE GIT ID: 77113c13de4ee973be8fe2df360f94e35356e6db. Plugins 6.1.x EE GIT ID: 2fbac957efca18f78f7db5354a61892098b4cb30.
          Hide
          Jonathan Lee added a comment -

          Need to re define the requirements. Not every link should dismiss the notification.

          Show
          Jonathan Lee added a comment - Need to re define the requirements. Not every link should dismiss the notification.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                2 years, 1 day ago

                Development

                  Structure Helper Panel