Base solution for your next web application
Open Closed

Please implement a single source of truth for logos #5861


User avatar
0
TimMackey created

Release 6.2.0 has the same logo names created in many components:

root.module.ts metaImage.setAttribute('content', window.location.origin + '/assets/common/images/app-logo-on-' + ui.getAsideSkin() + '.svg');

invoice.components.ts companyLogo = AppConsts.appBaseUrl + '/assets/common/images/logo.svg';

topbar.component.ts defaultLogo = AppConsts.appBaseUrl + '/assets/common/images/app-logo-on-' + this.ui.getAsideSkin() + '.svg';

UserNotificationHelper.ts icon: abp.appPath + 'assets/common/images/app-logo-small.svg',

default-brand.component.ts defaultLogo = AppConsts.appBaseUrl + '/assets/common/images/app-logo-on-' + this.ui.getAsideSkin() + '.svg';

I want to use .png file types in my app. Instead of having to change 5 files every upgrade cycle, it would be easier if all the logo names were consolidated in a single file.

Markdown is supported
Copy & paste or drag & drop images (max 30 MB per image)

3 Answer(s)
  • User Avatar
    0
    ismcagdas created
    Support Team

    Hi @timmackey

    • app-logo-on-dark.svg
    • app-logo-on-light.svg

    files are necessary for using black/white page headers and they are also necessary for some themes because some of Metronic themes uses 2 different logos.

    app-logo-small.svg is necessary for showing a browser notification to user when user got a notification.

    We can replace logo.svg with app-logo-on-dark.svg or app-logo-on-light.svg (I'm not sure which one is the correct one for now) but there will be still 3 logos instead of 4.

    Markdown is supported
    Copy & paste or drag & drop images (max 30 MB per image)
  • User Avatar
    0
    TimMackey created

    The number of logo files is not important. I'm requesting that the filenames be defined in the same file somewhere. Locating items that are likely to be customized to a single location would be preferrable to scattering them across the system.

    I've expended considerable effort designing my customized implementation of ANZ to minimize the impact of merging upgrades. Please consider what your customers may encounter for every upgrade cycle. Thank you.

    Markdown is supported
    Copy & paste or drag & drop images (max 30 MB per image)
  • User Avatar
    0
    ismcagdas created
    Support Team

    @timmackey This can be added to documentation, I have created an issue for this purpose https://github.com/aspnetzero/documents/issues/110

    Markdown is supported
    Copy & paste or drag & drop images (max 30 MB per image)