Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[44481]Expandable Enterprise Edition banners #11503

Merged
merged 7 commits into from
Nov 1, 2022

Conversation

bsatarnejad
Copy link
Contributor

@bsatarnejad bsatarnejad commented Oct 25, 2022

Make enterprise edition banner component extendible:
There is an input named collapsible, if you set it to true while using the component anywhere it will be extendible, if you don't set any value to it, it will work as before without collapsing button.

https://community.openproject.org/projects/openproject/work_packages/44481/activity

@bsatarnejad bsatarnejad self-assigned this Oct 25, 2022
@bsatarnejad bsatarnejad requested a review from a team October 26, 2022 04:54
<op-icon icon-classes="button--icon icon-medal"></op-icon> &nbsp;
<span [textContent]="text.enterpriseFeature"></span>
<div class="op-enterprise-banner--header">
<op-icon icon-classes="button--icon icon-medal"></op-icon>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

always use spot icons from now on

Suggested change
<op-icon icon-classes="button--icon icon-medal"></op-icon>
<span class="spot-icon spot-icon_medal"></span>

>
<span class="icon1"
[ngClass]="collapsed ? 'icon-arrow-down1' : 'icon-arrow-up1'"
aria-hidden="true">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same goes for thsi one, of course

this.collapsed = this.collapsible;
}

collapse():void {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd call this toggleCollapse or something since it works both ways.

@bsatarnejad bsatarnejad requested a review from b12f October 26, 2022 12:36
Copy link
Contributor

@b12f b12f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bsatarnejad
Copy link
Contributor Author

bsatarnejad commented Oct 27, 2022

Hi @marcalcobe, I replaced the new extendible EE banner with the previous one in any places that I thought it would look better, Could you please let me know if any change was not necessary and should be reverted? Or is there any place missing?

1- WP table view configuration:
wp-table-configuration

2- WP settings for highlighting:
wp-highlighting-mode

3- WP filters:
wp-filters

4- Boards filter:
board-filters

5- Projects settings:
projects-settings

6- New status definition in WP settings:
new-statu-swp-settings

7- Multi-select custom fields:
multi-select-custom-field

8- WP form configurations:
form-configurations

9- Custom fields for WPs:
custom-fields-wp-admin-settings

10- Custom fields for projects:
custom-fields-projects-admin-settings

11- Add widget modal:
add-widget-modal

@marcalcobe
Copy link
Contributor

Hi @bsatarnejad, sorry for the late response. I would suggest to not make it expandable in this cases as it is not really taking space from other content:

1- WP table view configuration
3- WP filters
4- Boards filter
11- Add widget modal

In regards to any missing page, I only can think of the new date alerts but I think that ones are tackled in a different PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants