Skip to content

Conversation

virzak
Copy link
Contributor

@virzak virzak commented Jul 6, 2025

This looks good on desktop, but not so much on mobile. Should probably be put on hold.

@virzak virzak marked this pull request as ready for review July 6, 2025 16:42
@michelebastione
Copy link
Contributor

I like that. We can add the colors back as well:

flowchart LR
    A1(["Excel analysis<br>process"]) --> A2{{"Unzipping<br>XLSX file"}} --> A3{{"Parsing<br>OpenXML"}} --> A4{{"Model<br>conversion"}} --> A5(["Output"])
    B1(["Other Excel<br>Frameworks"]) --> B2{{"Memory"}} --> B3{{"Memory"}} --> B4{{"Workbooks &<br>Worksheets"}} --> B5(["All rows at<br>the same time"])
    C1(["MiniExcel"]) --> C2{{"Stream"}} --> C3{{"Stream"}} --> C4{{"POCO or dynamic"}} --> C5(["Deferred execution<br>row by row"])

    classDef analysis fill:#D0E8FF,stroke:#1E88E5,color:#0D47A1,font-weight:bold;
    classDef others fill:#FCE4EC,stroke:#EC407A,color:#880E4F,font-weight:bold;
    classDef miniexcel fill:#E8F5E9,stroke:#388E3C,color:#1B5E20,font-weight:bold;

    class A1,A2,A3,A4,A5 analysis;
    class B1,B2,B3,B4,B5 others;
    class C1,C2,C3,C4,C5 miniexcel;
Loading

What do you think?

@virzak
Copy link
Contributor Author

virzak commented Jul 6, 2025

Yes, colored looks better.

@virzak
Copy link
Contributor Author

virzak commented Jul 6, 2025

Applied the changes. Not so sure if this should be bold font though. I'm cool either way.

🙌🙌🙌 Because in China Gitee platform not support mermaid yet, and lots miniexcel users in China use this platform : https://gitee.com/dotnetchina/MiniExcel 


Signed-off-by: Wei Lin <shps951002@gmail.com>
@virzak virzak merged commit 7e692bc into mini-software:master Jul 7, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants