ugent-library / biblio-backoffice

Apache License 2.0
7 stars 0 forks source link

Show legacy tag information in publication details – for librarians only #948

Closed mietcls closed 1 year ago

mietcls commented 1 year ago

Story

Show legacy tag information in publication details – for librarians only

Context

Old records that have missing information that cannot be recovered have a "legacy" tag. This is only in use for publications.

Current situation

This can be seen in:

Desired situation

The information is visible in the detail page as well:

Detail page: Publication details

In the detail page when not editing it.

Current view: Image

Desired view:

Image

Implementation suggestion

<div class="form-group row">
    <label class="col-lg-3 col-form-label align-self-start">Legacy record</label>
    <div class="col-md-9 mt-3">
        <i class="if if-check-circle-fill"></I>
    </div>
</div>

Detail page: header

947

nicolasfranck commented 1 year ago

your suggestion formats a boolean differently (e.g. "extern", commonly known as "published when not working at ugent", is simply a ✓). Keep everything in line?

mietcls commented 1 year ago

I'd like to make sure we use Emily's icons, if that's what you're referring to? I was going to still make an issue or adapt it myself for the "extern" boolean – if I don't break anything because I currently can't run it locally.

nicolasfranck commented 1 year ago

well there aren't so many cases where such a flag exists. Only "extern", I believe. We cannot have an icon on one place, and a bit of text elsewhere. Easy to add too. I would use the close-circle-fill for the "false" value?

mietcls commented 1 year ago

@nicolasfranck for the false value please keep the dash, I added the circle for contrast reasons. When you see the circle you immediately know what it is without having to take a second look or squint. True = if-check-circle-fill False = -

If you can also align it for "extern" that would be great.