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

[Object Page Component]: facets design inconsistencies #5811

Closed
1 task done
ShirSAP opened this issue May 16, 2024 · 4 comments
Closed
1 task done

[Object Page Component]: facets design inconsistencies #5811

ShirSAP opened this issue May 16, 2024 · 4 comments
Labels
author action bug Something isn't working SAP Build

Comments

@ShirSAP
Copy link

ShirSAP commented May 16, 2024

Bug Description

Today in the object page facet area, it seems that the display mode form are not using the correct parameters and padding.

This is how the component shows up without custom coding in our system:
image
Both the form label as well as the paddings are incorrect.

Affected Component

https://sap.github.io/ui5-webcomponents-react/?path=/docs/layouts-floorplans-objectpage--docs

Expected Behaviour

Form Label should use:
image
Form text value should use:
image

Taken from the following wiki: https://wiki.one.int.sap/wiki/pages/viewpage.action?pageId=2697896853

In vertical layout the padding between label and text value should be:
image

Taken from the following wiki: https://wiki.one.int.sap/wiki/pages/viewpage.action?pageId=2698913658

Isolated Example

No response

Steps to Reproduce

...

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

1.18.0

Browser

Chrome

Operating System

OS

Additional Context

No response

Organization

SAP Build

Declaration

  • I’m not disclosing any internal or sensitive information.
@ShirSAP ShirSAP added the bug Something isn't working label May 16, 2024
@ilhan007 ilhan007 transferred this issue from SAP/ui5-webcomponents May 16, 2024
@ilhan007
Copy link
Member

Hello @ShirSAP and colleagues
there is no Object Page in UI5 Web Components project, the links from the issue description are from the UI5 Web Components For React Project => transferring the issue to https://github.com/SAP/ui5-webcomponents-react repository.

@ShirSAP ShirSAP changed the title [Object Page facets design inconsistencies]: [Object Page Component]: facets design inconsistencies May 16, 2024
@Lukas742
Copy link
Contributor

Lukas742 commented May 16, 2024

Hi @ShirSAP

although the screenshots you posted don't contain any sensitive information, please be aware that this is a public GitHub repository and that we generally delete content that we think might be compromising or just shouldn't be made public. I only mention this to raise awareness - it's always better to be safe than sorry.

This is how the component shows up without custom coding in our system:

The DynamicPageHeader accepts any content developers define, so without a reproducible example I can't really tell what components you used. Please add an isolated, reproducible example by using this StackBlitz template, so we can assist you further.

Also, we're currently not offering any "Facet" components, so it's up to developers to define them on their end. If you feel like one of these facets should be available as component, please create a feature request for the respective component.

In vertical layout the padding between label and text value should be:

Since you have linked the specifications of the Form component here, I have checked them and found no inconsistencies with our implementation in the vertical layout. If I have missed something, please indicate specifically what is not implemented according to the guidelines. Please note that only the visual aspect has to comply with the guidelines, e.g. it doesn't matter if we use gap instead of padding, it just has to reflect the values defined in the wiki.

@ShirSAP
Copy link
Author

ShirSAP commented May 16, 2024

Hello @Lukas742,

Appreciate your awareness comment and super quick response 😇
I've requested the support of a developer in providing a StackBlitz example for you.

Hopefully he could get this to us soon.

@ShirSAP
Copy link
Author

ShirSAP commented May 23, 2024

Hello @Lukas742
My development team got back to me, it seems that although the screenshot provided above was said to be without any custom coding, there were still a little left.

In the following example you'll see that everything looks good.
I'll be closing this ticket.

Thank you and my apologies for the confusion.

@ShirSAP ShirSAP closed this as completed May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author action bug Something isn't working SAP Build
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants