55 Commits

Author SHA1 Message Date
Ghostkeeper
7529483cb0
Don't show download count for bundled plug-ins
I considered rewriting the section title property to be QML-only and translate it from the QML, but this is a bit simpler in the end, even though there is data duplication now.

Contributes to issue CURA-8565.
2021-11-30 14:42:12 +01:00
Ghostkeeper
cac623b509
Use medium font size for extended description
The design appears more balanced then, quoth the designer of the layout.

Contributes to issue CURA-8565.
2021-11-30 10:49:49 +01:00
Ghostkeeper
f385e3d639
Don't show elision or read more if there is no description at all
Apparently Qt marks it as 'truncated' then, even though it's not really. Don't show the ... nor the 'read more' button if there is nothing more to read.

Contributes to issue CURA-8565.
2021-11-29 19:59:43 +01:00
Ghostkeeper
d96284ee3e
Only show formatted description in detail page
We can't show rich text in the package list, because the we use the onLineLaidOut signal there, which doesn't work with Rich Text. So for the package list we should NOT use the formatted version of the description because that will contain ugly HTML tags that the user wouldn't want to see. Show the original description there. Use the formatted description only in the detail page where we don't use onLineLaidOut.

Contributes to issue CURA-8565.
2021-11-29 19:35:57 +01:00
Ghostkeeper
b5c7dfe9a2
Format links in descriptions to be clickable
Took some fiddling to get the regex right. But it's nice now.

Contributes to issue CURA-8565.
2021-11-29 19:29:33 +01:00
Ghostkeeper
ffd1a4d812
Add button to visit plug-in website
There are a lot of buttons leading to websites now: An arrow leading to the author website. An author name leading to the author website. A 'read more' label leading to the plug-in website and this new button leading to the plug-in website. Maybe we should raise this with the designer.

Contributes to issue CURA-8565.
2021-11-29 18:46:56 +01:00
Ghostkeeper
e173fa2d9d
Add package description in full to detail card
I realise that this might cause the card to become taller than the window. Might need to do something about that.

Contributes to issue CURA-8565.
2021-11-29 18:33:58 +01:00
Ghostkeeper
0546f58e57
Add extended description header
Wrapping the whole content so far in another column so that we can have a wider part below, automatically aligned.

Contributes to issue CURA-8565.
2021-11-29 18:26:28 +01:00
Ghostkeeper
cfd29b268d
Use actual download count for packages
Quite easy. It turned out the model already had this information, due to foresight when the card itself got implemented.

Contributes to issue CURA-8565.
2021-11-29 18:05:53 +01:00
Ghostkeeper
0dcc28032a
Add download count design to detail card
Just the design. The data is just a placeholder so far.

Contributes to issue CURA-8565.
2021-11-29 18:03:42 +01:00
Ghostkeeper
443ba67455
Align ellipsis to text again
It used to be fine, but the font of the button got made different from the font of the description. It should be the same as the description, so this should remain correct even if the fonts change again.

Contributes to issue CURA-8565.
2021-11-29 17:49:11 +01:00
Ghostkeeper
d7058ef520
Change card summary information into a column
This is necessary because we can't anchor to the bottom of the card here, and because we want to swap out the description for a download count in the extended detail card.

Contributes to issue CURA-8565.
2021-11-29 17:40:53 +01:00
Ghostkeeper
8ecd2f86a4
Communicate to PackageCard whether it is a detailed card or not
If it is detailed, it currently hides the short description. That is not quite enough, but we'll expand that behaviour.

Contributes to issue CURA-8565.
2021-11-29 16:38:24 +01:00
Ghostkeeper
616bf479e5
Set position and width of card in details page
This means that the card itself shouldn't specify a width. It should get a width from how it's used.

Contributes to issue CURA-8565.
2021-11-29 16:11:15 +01:00
Jaime van Kessel
09a569fdd1 Change fonts to default
No idea why they were medium, but the design clearly shows that it should be the default font
2021-11-26 14:39:26 +01:00
Jaime van Kessel
60b174177f Also set correct size for package card title 2021-11-26 14:39:02 +01:00
Ghostkeeper
bddcf3cb0c
Add extensible switch case for tooltip
This way it's
- more clear what the two cases are
- and more robust if we ever add a third.

Contributes to issue CURA-8562.
2021-11-22 17:16:39 +01:00
Jelle Spijker
7fd9578599
Unified tooltip description
Contributes to CURA-8562
2021-11-19 11:21:29 +01:00
Jelle Spijker
eb3083c84d
use differentiate icon for Ultimaker controlled material/plugin
Materials can be certified, while plugins can be verified.
Added packageType to the model such that the card knows which icon
to use.

Contributes to CURA-8562
2021-11-19 11:21:29 +01:00
Jelle Spijker
584411e59e
Moved comments
if they're inline they mess up the QML syntax highlighter in PyCharm
2021-11-19 11:21:21 +01:00
Remco Burema
dcb9456848
Fix CApitalization and spurious space.
part of CURA-8561
2021-11-18 08:44:06 +01:00
Jaime van Kessel
07e9237d72 Add missing action for button
CURA-8561
2021-11-17 14:23:26 +01:00
Jaime van Kessel
183fa06cfe Fix look of external link button
CURA-8561
2021-11-17 11:54:12 +01:00
Jaime van Kessel
5ac0df8b0f Fix styling of scrollbar in marketplace
CURA-8561
2021-11-17 11:22:30 +01:00
Jaime van Kessel
f01ce5b43c Fix direction of tooltip point
CURA-8561
2021-11-17 10:59:31 +01:00
Jaime van Kessel
d7e023c5ee Fix sizes of icons in package card
CURA-8561
2021-11-17 10:56:06 +01:00
Jaime van Kessel
4ae01df7f5 Ensure that the packagecard looks like the design
CURA-8561
2021-11-17 10:27:48 +01:00
Jaime van Kessel
c8491b4752 Remove background color from external link button
CURA-8561
2021-11-17 09:15:08 +01:00
Jaime van Kessel
980cc22529 Simplify the layout of the packageCard
CURA-8561
2021-11-16 18:02:56 +01:00
Jaime van Kessel
e44a58b3a3 Use reusable simple button instead of redefining it 2021-11-16 16:51:09 +01:00
Remco Burema
7c29e69f55
Small UX adjustments.
part of CURA-8561
2021-11-16 16:08:29 +01:00
Remco Burema
d47b2fb5dd
Control should be (in)visible, not (just) image.
part oc CURA-8561
2021-11-12 08:52:48 +01:00
Remco Burema
4a7a74cba6
Also make description area foldable.
part of CURA-8561
2021-11-12 08:45:09 +01:00
Remco Burema
82f140aa39
Folded versus unfolded.
Hide disable/uninstall/install buttons, they're not active anyway and it's not part of this ticket in what state they should be hidden or not. What is part of the folded versus header is the download count row. (Also adapt link color.)

part of CURA-8561
2021-11-11 18:07:21 +01:00
Remco Burema
c1f2da8820
Layout fixes.
Prevent 'height' based binding loops.

part of CURA-8561
2021-11-11 17:30:43 +01:00
Remco Burema
51de234082
Links, hovers, ensmallify layout.
part of CURA-8561
2021-11-10 18:09:36 +01:00
Ghostkeeper
1efdd9205b
Use primary colour for verified icon
Contributes to issue CURA-8561.
2021-11-09 17:47:32 +01:00
Ghostkeeper
d186912596
Correcter font sizes according to design
Contributes to issue CURA-8561.
2021-11-09 17:45:44 +01:00
Ghostkeeper
d526e3be8c
Easier layout shifting when icons are invisible
The Row element automatically hides them and removes any spacing if they are invisible.

Contributes to issue CURA-8561.
2021-11-09 17:40:10 +01:00
Ghostkeeper
f498952830
Give text colours to text
This way it's still visible in the dark theme, rather than black on black.

Contributes to issue CURA-8561.
2021-11-09 17:32:22 +01:00
Ghostkeeper
4119cf4209
Fix layout of author button
Bold, correct position and add the icon.

Contributes to issue CURA-8561.
2021-11-09 17:30:52 +01:00
Ghostkeeper
234475547e
Use correct font for all text elements
Contributes to issue CURA-8561.
2021-11-09 17:25:02 +01:00
Ghostkeeper
c56240f276
Use correct icon for external links
Contributes to issue CURA-8561.
2021-11-09 17:22:01 +01:00
Ghostkeeper
5a698bd91f
Truncate double ellipsis where possible
I couldn't get it to truncate it if the double ellipsis is the only text on the line, like if the description contains a white line and more than 2 lines in total. It then looks like a double ellipsis (6 dots instead of 3). Doesn't look the worst, but a bit strange, but it's really difficult to fix.

Contributes to issue CURA-8561.
2021-11-09 17:20:37 +01:00
Ghostkeeper
7b7cb43b02
Improved elision
It seems to correctly place the elide character now. One more detail that's incorrect is that it shows two elision characters if it's eliding due to maximum line count. I'll see what I can do...

Contributes to issue CURA-8561.
2021-11-09 16:58:30 +01:00
Ghostkeeper
57093f0ef6
Hide Read More button if not truncated
There would be nothing to read.

Contributes to issue CURA-8561.
2021-11-09 16:34:21 +01:00
Ghostkeeper
c8741898bf
Decent attempt at inlining Read More button
Not perfect yet. The elide is missing, for one.

Contributes to issue CURA-8561.
2021-11-09 16:28:12 +01:00
Ghostkeeper
468c2b89e1
Use wrapping for package description
Contributes to issue CURA-8561.
2021-11-09 16:09:58 +01:00
Ghostkeeper
8c086b9fd7
Align everything to top
This seems to be the alignment in the design. Also gets rid of binding loops because we automatically adjust the height so you can't align to the centre or the bottom then.

Contributes to issue CURA-8561.
2021-11-09 15:58:37 +01:00
Ghostkeeper
abe8347523
Fix QML warning about not having parents
It seems that the ListView doesn't always set the parent element correctly if it's not yet in view. This is a workaround that seems to work fine to remove the QML warnings about parent not being defined.

Contributes to issue CURA-8561.
2021-11-09 15:54:42 +01:00