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

DRAFT - Improve image cropping drawer #73

Open
Julien-Dubois-eXo opened this issue Jul 31, 2023 · 5 comments
Open

DRAFT - Improve image cropping drawer #73

Julien-Dubois-eXo opened this issue Jul 31, 2023 · 5 comments

Comments

@Julien-Dubois-eXo
Copy link

Rationale
Details of the GAP ( problem(s) to address ) expressed by few words using some screenshots.
Note: avoid describing the solution here.

Currently, we have an issue when selecting an image into the News feature (specific to eXo) to properly address the display of the image in case of multiple screen sizes. The user has an issue knowing what the image will look like in the different size context.

We want to use the image cropping component to be able to manage the size and be able to crop propose a tool answering the need. So we could propose a harmonized experience using the component already used.

Additionally, following the rework of the login page, we can't display an illustration in the mobile version as the size are different:

  • desktop: illustration is a vertical rectangle
  • mobile: illustration should be a horizontal rectangle.

The solution described below may allow us to have an illustration on mobile too.

  1. Functional requirements

Add in the cropping components additional option to be able to manage different layouts of images in a responsive context:

  • A new section "Image layout" (extensibility should be implemented to be able to manage the name of the section in the different integration of the component)
  • An descriptive text to explain what is the purpose of this option
  • 4 layout selectors:
    • square
    • rectangle
    • large rectangle
    • vertical rectangle
  • When downloading a picture, it's displayed with the cropping frame
  • When switching from one layout to another we can visualize and adapt the image to the selected layout
  • "Multiple image" option: allow to download a different image for each layout so if the image is can't be adapted to a specific layout we can use another one.
  • We will use a ratio to define which layout should be used so we could efficiently manage the responsiveness of the page.

Image
Image

  1. Non Functional Requirements

Expected volume & Performance requirements

Security requirements (Authentication, Authorizations, Audit track / traceability, data retention…)

Extensibility requirements (Identified extension points, plugables implementations...)

Configurability requirements (properties introduced, default settings, jobs, etc...)

Requirements related to existing users data

Requirements related to existing features (ex: Add a button to existing modules...)

Feature Flag (property name, default status, target audience, functional behaviour)

Other NFR (ex: Use of VueX, Vue, Vuetify)

  1. Impacts

Requirements (Squash)

Documentation to update?

Training level / Change management needs

  1. Software Architecture
  • Security

  • Access (GUI, API…)

    • Rest API design (Entry points, domain objects)
    • Portlet Vue & Vuetify, simple portlet, extension WAR ...
  • Services & processing

    • Ex: Scheduled Jobs
  • Data and persistence

    • Macro data model
    • Persistence layer (RDBMS, elasticsearch...)
    • Use of caches
  • Support for clustering

  • Support for multitenancy

  • Integrations

  • Migration strategy

  1. Annexes
    • Reasoning that explain the technical choices
    • Impacts (ex: impact on public REST/Java API)
    • References (links...)
@Julien-Dubois-eXo Julien-Dubois-eXo converted this from a draft issue Jul 31, 2023
@Julien-Dubois-eXo
Copy link
Author

@srenault-meeds to be discussed after the holiday break, please.

@srenault-meeds
Copy link
Member

Hi, it seems it is completely different to Meeds usage and if you allow me, it seems to be very specific to eXo usage.
However, opened to discuss it anytime

rdenarie added a commit to exoplatform/ecms that referenced this issue Oct 16, 2023
rdenarie added a commit to exoplatform/web-conferencing that referenced this issue Oct 16, 2023
rdenarie added a commit to exoplatform/ecms that referenced this issue Oct 16, 2023
rdenarie added a commit to exoplatform/web-conferencing that referenced this issue Oct 16, 2023
@srenault-meeds
Copy link
Member

Hi,
Is that still something planned to be done?

@Julien-Dubois-eXo
Copy link
Author

Not for 6.5. For later I have not the visibility.

@srenault-meeds
Copy link
Member

I Archive it as we don't have any news about it. If needed, a restore can be done

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

No branches or pull requests

2 participants