Redesign: contact Row and Picture: themeing and modularizing: #276

Merged
sarah merged 2 commits from dan/ui:03-widgets into master 2 years ago
dan commented 3 years ago
Owner

Redesign: contact Row and Picture: themeing and modularizing:

  • Picture was split into Badge, and Portrait
  • ContactRow was split into: PortraitRow, ProfileRow, and ContactRow
  • Removed some unsued cruft
  • Still need to work on adopting new images (requires coloring b/w images
    to theme colors...) And adding the new accept/reject new contact
    workflow
Redesign: contact Row and Picture: themeing and modularizing: - Picture was split into Badge, and Portrait - ContactRow was split into: PortraitRow, ProfileRow, and ContactRow - Removed some unsued cruft - Still need to work on adopting new images (requires coloring b/w images to theme colors...) And adding the new accept/reject new contact workflow
erinn was assigned by dan 3 years ago

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/523

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/523
Owner
  • badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle
  • if (status == 4 || status == 3) ... else if (status == 3 || status == 2) is redundant
  • some things in the attached screenshot:
    • long profile names and onion strings cross the scrollbar
    • the profile images in the message pane seem to be of varying sizes for some reason?
    • profile images in the message pane have an empty white badge
  • in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol
* badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle * if (status == 4 || status == **3**) ... else if (status == **3** || status == 2) is redundant * some things in the attached screenshot: * long profile names and onion strings cross the scrollbar * the profile images in the message pane seem to be of varying sizes for some reason? * profile images in the message pane have an empty white badge * in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol
Poster
Owner

awesome, I'll dig into this

responses

  • badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle

asking marcia
if so: textMetric code to be put back in... but width is decoupled, the filler content is in contactRow and badge width initially set in Badge... so that'll be something for me to figure out

  • if (status == 4 || status == 3) ... else if (status == 3 || status == 2) is redundant

fixed
-_-;

some things in the attached screenshot:

  • long profile names and onion strings cross the scrollbar

    • existing behaviour but more noticable now with larger fonts
      TODO
      • resize min width of bar to at least fit onion address
      • use ?textMetric to ellipsise text before spilling off
  • the profile images in the message pane seem to be of varying sizes for some reason?

    • thats down to how they are being invoked by the message pane. I was thinking that out of scope for this PR, as this is def a WIP and I was thinking to sort that out when I did a full message pane treatment. should have called it out, is ok?
  • profile images in the message pane have an empty white badge

    • ditto
  • in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol

    • I can take a look and prep a seperate PR where I run one of them over the code + a tidy.sh script to keep it going, gooooood call out!
awesome, I'll dig into this responses - badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle asking marcia if so: textMetric code to be put back in... but width is decoupled, the filler content is in contactRow and badge width initially set in Badge... so that'll be something for me to figure out - [x] if (status == 4 || status == 3) ... else if (status == 3 || status == 2) is redundant fixed -_-; some things in the attached screenshot: - long profile names and onion strings cross the scrollbar - existing behaviour but more noticable now with larger fonts TODO - [ ] resize min width of bar to at least fit onion address - [ ] use ?textMetric to ellipsise text before spilling off - the profile images in the message pane seem to be of varying sizes for some reason? - thats down to how they are being invoked by the message pane. I was thinking that out of scope for this PR, as this is def a WIP and I was thinking to sort that out when I did a full message pane treatment. should have called it out, is ok? - [ ] profile images in the message pane have an empty white badge - ditto - in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol - I can take a look and prep a seperate PR where I run one of them over the code + a tidy.sh script to keep it going, gooooood call out!
Poster
Owner

if (status == 4 || status == 3) ... else if (status == 3 || status == 2) is redundant

fixed

long profile names and onion strings cross the scrollbar

now elipsis dynamically

badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle

badge dimensions stay fixed but as per marcia, number caps at string "99+" which fits

the profile images in the message pane seem to be of varying sizes for some reason?
profile images in the message pane have an empty white badge

To be addressed in message pane PR

in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol

scheduling for seperate PR cus... all the red/green as all the whitespace changes

> if (status == 4 || status == 3) ... else if (status == 3 || status == 2) is redundant fixed > long profile names and onion strings cross the scrollbar now elipsis dynamically > badge dimensions aren't dynamic anymore... long numbers eg 999 will overflow out of the badge circle badge dimensions stay fixed but as per marcia, number caps at string "99+" which fits > the profile images in the message pane seem to be of varying sizes for some reason? > profile images in the message pane have an empty white badge To be addressed in message pane PR > in general, not necessarily for this PR: this diff was somewhat hard to review because our indentation is getting messier over time... can we get a formatting tool involved? would either https://github.com/jesperhh/qmlfmt or https://github.com/Orange-OpenSource/qmljsreformatter work with our process? both annoyingly appear to replace tabs with spaces but i'm willing to trade one pain for the other in this case lol scheduling for seperate PR cus... all the red/green as all the whitespace changes

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/524

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/524
Poster
Owner
There is no content yet.
sarah closed this pull request 2 years ago
This repo is archived. You cannot comment on pull requests.
Loading…
There is no content yet.