Skip to content

Docs/ai chat history#1636

Draft
panch1739 wants to merge 4 commits into
mainfrom
docs/ai-chat-history
Draft

Docs/ai chat history#1636
panch1739 wants to merge 4 commits into
mainfrom
docs/ai-chat-history

Conversation

@panch1739

@panch1739 panch1739 commented Mar 13, 2026

Copy link
Copy Markdown
Member

This PR adds documentation to construct the AI chat history.

Please make sure your PR follows the contribution guidelines
https://github.com/siemens/element/blob/main/CONTRIBUTING.md. -->

Dependency with: #1641

Documentation.
Examples.
Dashboards Demo.
Playwright report.

Coverage Reports:

Code Coverage

@panch1739 panch1739 added this to the 49.x milestone Mar 13, 2026
@panch1739
panch1739 requested a review from hbxes March 13, 2026 10:05
@panch1739 panch1739 added documentation Improvements or additions to documentation ux Marks all UX related topics (UX team is working on or UX input is required) labels Mar 13, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces documentation for the AI chat history pattern. The new documentation is well-structured and informative. I have provided a couple of suggestions to better align the content with the repository's UX writing guidelines, specifically concerning sentence length and image accessibility.

Comment thread docs/patterns/ai/ai-chat-history.md
Comment on lines +50 to +51
If space remains insufficient due to a high number of metadata elements, consider prioritizing
the most relevant attributes and progressively removing secondary ones to preserve clarity.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This sentence is 30 words long, which exceeds the 25-word limit recommended by the repository's UX writing guidelines for better readability. Please consider shortening it.

Suggested change
If space remains insufficient due to a high number of metadata elements, consider prioritizing
the most relevant attributes and progressively removing secondary ones to preserve clarity.
If space is insufficient, prioritize the most relevant attributes and progressively remove secondary ones to preserve clarity.
References
  1. The repository's UX Writing Guidelines recommend keeping sentences under 25 words for readability. (link)

@hbxes

hbxes commented Mar 13, 2026

Copy link
Copy Markdown
Member

What happens if you click on the icon below??
Bildschirmfoto 2026-03-13 um 12 17 29

@hbxes

hbxes commented Mar 13, 2026

Copy link
Copy Markdown
Member

otherwise, it's good to go...

@panch1739

Copy link
Copy Markdown
Member Author

What happens if you click on the icon below?? Bildschirmfoto 2026-03-13 um 12 17 29

The icon is read-only, like ix has it. We had a chat with @spike-rabbit to check the implementation.

@hbxes

hbxes commented Mar 23, 2026

Copy link
Copy Markdown
Member

What happens if you click on the icon below?? Bildschirmfoto 2026-03-13 um 12 17 29

The icon is read-only, like ix has it. We had a chat with @spike-rabbit to check the implementation.

I still don't get it, as a user I want to click on it but then nothing happens, it seems, that it's a "Title" on what you can expect when you open the side panel... I mean if iX has it, it's not a a proof of quality... I think we should reconsider it, also what if there are various other content in the side panel, what do we show then...

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

Labels

documentation Improvements or additions to documentation ux Marks all UX related topics (UX team is working on or UX input is required)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants