Apache Allura™

Hi..

I have added the user card (Please check the attachment for the preview). For now I have added display_name, city, country with an icon, avatar image. What else we can add into the user card. User card will be attached to the links which have user-mention css class and it will use href to catch the user.

Btw. I am getting a lot of test case failures with the latest commit from the upstream. Is there an issue?

Please advice. Thanks

I think including the user's webpage could be nice.

The default content of content: 'test' shows up briefly, while its loading. Maybe make that empty, or 'Loading...'

I think it'd be better to use User.icon_url method instead of g.gravatar, since it will also support locally uploaded profile images.

Constructing HTML directly in JS isn't ideal, as it can be a risk for XSS injection attacks, and this code is. A templating language like Handlebars could be used instead, but we don't have any in place in Allura yet. (And handlebars can be annoyingly restrictive sometimes). I'm not finding any real templating or lightweight frontend library in Allura yet, which is too bad. The best I can find as a better recommendation is to use jquery to call .text(...) and .attr(... Here's a random example I found, to illustrate:

      $('<span>')
        .text('▼')
        .attr('tabIndex', -1)
        .attr('title', 'Show all options')
        .appendTo(wrapper)
        .addClass('ui-combobox-toggle')
        .click(openDropdown);

Perhaps even better, although it changes the implementation a fair bit, would be to have the /user_card URL endpoint return HTML using a server-side jinja template. Then you get the templating & escaping from Jinja. And the JS would just cache + tooltip that html directly. I think that would be the best way to go.

If in the future there's some need to build it in JS, like if was going to be more dynamic & changing (although I can't imagine what that would be) then we could figure that out then. We'd want to add a good JS library for that into Allura. @kentontaylor and I have been pretty interested in Svelte lately; its a very lightweight frontend framework.

Hi..

Yes I also agree, returning HTML using jinja is the best choice for now. I will update with required changes

Thanks

Hi.. @brondsem I've updated with following modifications

• Used jinja html template for user card (Added Apache license header)
• Changed json endpoint to html
• Displayed website if set
• Updated unit test cases
• Added Loading... text to user card

Thanks

Looks good now!