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

new: Promo/flag/ribbon/special marker #48

Closed
jackreeves opened this issue Jun 14, 2018 · 22 comments
Closed

new: Promo/flag/ribbon/special marker #48

jackreeves opened this issue Jun 14, 2018 · 22 comments
Assignees
Labels
priority: medium medium priority item status: completed 💥 issue completed status: DPA design approved approved design by the DPA that can be picked for development type: new component 🎁 New component

Comments

@jackreeves
Copy link
Contributor

jackreeves commented Jun 14, 2018

Problem Statement

as a customer i would like a clear way of seeing if something is on sale or a special feature or new so that i can focus my attention

as a designer i would like a way for all teams to consistently approach flags so that we can improve user experience

Acceptance Criteria:

  • Ribbon should have margin of 16px above and below.
  • Should have inner padding of 8px top and bottom.
  • The ribbon should expand and contract to fit the inputted text.
  • Any text should be allowed as an input / prop, but should be limited to strings only.
  • There should only be one line of text allowed, must not wrap to two lines.
  • The minimum total width of the ribbon is 126px.

Recommendation

  • Description of the proposed solution

Designs

screen shot 2018-06-14 at 3 18 05 pm
screen shot 2018-06-14 at 3 17 45 pm

Meta

  • Willing to design solution: No
  • Willing to develop solution: No
  • Has workaround: No
  • Do any similar components already exist: No
  • Does this pattern exist already but isn’t a community component?
    https://www.telus.com/en/bc/mobility/phones/?linktype=nav
  • Which teams/applications do you think would use this improvement?
    MOB/HS/B2B
  • High impact: No
@jackreeves jackreeves added this to To do in Community Backlog via automation Jun 14, 2018
@varunj90 varunj90 moved this from To do to Design intent in Community Backlog Feb 4, 2019
@varunj90 varunj90 moved this from Design intent to Design queue in Community Backlog Feb 4, 2019
@mike-bunce
Copy link
Contributor

Our team (HS R+L) will be building this component as we have a requirement for it in our current sprint.

@twirlingsky
Copy link

twirlingsky commented Feb 25, 2019

Home Solutions Ribbon:

Usage criteria: for use on a product card to signify a term length contract on an offer (e.g. 2 year term)

must:

  • Telus purple for ribbon colour
  • white text colour, medium weight
  • follow product / offer name
  • be above price block (in eyebrow position)

screen shot 2019-02-25 at 10 13 17 am
screen shot 2019-02-25 at 10 23 21 am

@varunj90
Copy link
Contributor

@sketchidea are you/ your team still planning to develop this component?

@nicmak
Copy link
Contributor

nicmak commented Mar 26, 2019

Hi Everyone, our designer Francisco has new designs @twirlingsky
ribbons

@varunj90 varunj90 moved this from Design queue to Design review in Community Backlog Mar 29, 2019
@donnavitan
Copy link
Contributor

donnavitan commented Apr 1, 2019

Connected with Francisco regarding "ribbon".

Please review the following file: https://invis.io/a/HJ5CGILLKWQT
Please make sure that you sync/update the file here: https://telus.invisionapp.com/d/main#/projects/prototypes/13362811/assets

I've done the following:

  • Height: 32px
  • Minimum Width: 131px
  • Updated typography rules (it should align with existing text/medium/* styles (but now available in medium and inverted/white)
  • 16px at 24px line-height
  • Left aligned
  • It has a left padding of 16px
  • It has a right padding of 24px (including the 8px of tail)
  • Vertically centered visually (inspect Invision for reference) https://telus.invisionapp.com/share/5TRB1UFAD4U
  • Created it as a symbol so all you have to do is override the label
  • Will automagically expand (Anima plugin required)

Next steps:

  • Review as necessary
  • Add "ribbon" symbol to the TDS Community DSM library under "Components (not coded)"
  • Confirm design/dev
  • Add usage criteria (for DSM library)

@donnavitan
Copy link
Contributor

Oh! If it's released in code, then:

  • Add "ribbon" symbol to the TDS Community DSM library under "Components (coded)"

@varunj90 varunj90 added status: DPA design review needed A feature or request that requires DPA design review and removed status: design needed 🎨 labels Apr 1, 2019
@varunj90 varunj90 moved this from Design review to In development (code and design) in Community Backlog Apr 1, 2019
@donnavitan
Copy link
Contributor

donnavitan commented Apr 5, 2019

Hello folks, as per my discussion with Francisco, I've added the ribbon component in TDS Community DSM library under "Components (not coded)". When dev is completed, we can move to "Components (coded)". Cheerios!

https://telus.invisionapp.com/dsm/telus/tds-community/folder/components/5ca657910cb8d13e823259f7

@varunj90 varunj90 added status: DPA design approved approved design by the DPA that can be picked for development and removed status: DPA design review needed A feature or request that requires DPA design review labels Apr 5, 2019
@mike-bunce
Copy link
Contributor

Acceptance criteria added

@nicmak
Copy link
Contributor

nicmak commented Apr 17, 2019

Starting Development today

nicmak added a commit that referenced this issue Apr 17, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
@varunj90 varunj90 assigned Franimal604 and mike-bunce and unassigned donnavitan Apr 22, 2019
nicmak added a commit that referenced this issue May 6, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
nicmak added a commit that referenced this issue May 7, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
nicmak added a commit that referenced this issue May 17, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
nicmak added a commit that referenced this issue May 17, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
nicmak added a commit that referenced this issue May 22, 2019
adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48
ghost pushed a commit that referenced this issue May 28, 2019
* new lock file

* feat(community-ribbon): adding new ribbon component to tds-community

adding ribbon component for clear way for customers to see sale or special feature items

new: Promo/flag/ribbon/special marker #48

* fix(community-ribbon): changes to code according to PR

updating usage criteria

* fix(community-ribbon): changes to code according to PR

updating usage criteria

* fix(community-ribbon): snapshots and e2e tests

updating usage criteria along with tests

* fix(community-side-navigation): updating snapshots and e2e tests

updating tests

* fix(community-ribbon): updating tests

:Skip

* fix(community-side-navigation): updating e2e tests

* fix(community-pagination): updating e2e tests

* fix(community-testimonial): updating e2e tests

* fix: Update e2e shots

* fix(community-ribbon): making style adjustments

* new e2e test

* fix(community-ribbon): removing props in jest test

:sjip
@donnavitan
Copy link
Contributor

Adding notes for reference:

@donnavitan
Copy link
Contributor

@twirlingsky Could you confirm if the left padding should be changed to 8px or keep 16px? Did you want to keep the 24px right padding? Did you want to keep a minimum width or just have the ribbon background expand to whatever the label is?

@twirlingsky
Copy link

I just gave it a double check and posted a new symbol to DSM.

Padding left: 8px
Padding right: 16 + 8px - 24 px to allow for the ribbon tail
Padding top and bottom: 4px
Minimum width of 126px is still on - the background can expand to whatever the label is

@donnavitan
Copy link
Contributor

donnavitan commented Jun 10, 2019

Hello all,

As per @twirlingsky's confirmation, here's an updated specs and examples:

Details

  • Height: 24px
  • Minimum Width: 126px
  • Font: Helvetica Neue
  • Font-size: 16px
  • Line-height: 16px
  • Weight: Medium (aka bold)
  • Align: Left
  • Colour: White #FFFFFF (aka inverted)
  • Vertically centered visually (inspect Invision for reference)
  • Left padding: 16px
  • Right padding: 24px (including 8px of ribbon tail)
  • Top margin: 8px
  • Bottom margin: 8px

Sketch use

  • Use override panel to edit the label
  • Will automagically expand (Anima plugin required)

image

https://telus.invisionapp.com/share/5TRB1UFAD4U

TDS Community DSM library has been updated accordingly.

@donnavitan
Copy link
Contributor

Please note that the latest working file for this is: https://invis.io/a/HJ5CGILLKWQT

@donnavitan
Copy link
Contributor

@mejsullivan
Copy link

Are their docs to see what's been done for accessibility? Saw this was getting released! Exciting!

@donnavitan
Copy link
Contributor

@mejsullivan Thank you for bring that up. Reaching out to you and @twirlingsky for additional context where applicable.

It's not an interactive element, but it is a visual callout - so maybe there is an opportunity to enhance the a11y experience. Unfortunately, the use is so free form that it's not limited to "sale", "offer", or "program". Maybe, the only addition is to include such term in the ribbon label?

Please advise and recommend, if folks have suggestions.

@varunj90 varunj90 moved this from In development (code and design) to In QA in Community Backlog Jun 12, 2019
@donnavitan
Copy link
Contributor

Just adding a note here that design a11y review has been double checked. While the opportunity to enhance the component by providing more meaningful context can be an enhancement; to be considered with @twirlingsky and design direction/CX.

@varunj90 varunj90 added the status: completed 💥 issue completed label Jun 13, 2019
@varunj90 varunj90 moved this from In QA to Released in Community Backlog Jun 13, 2019
@varunj90 varunj90 assigned ghost Jun 13, 2019
@mejsullivan
Copy link

mejsullivan commented Jul 4, 2019

@varunj90 @donnavitan - sorry I don't understand what was said above. Is it accessible? y/n? :)

@donnavitan
Copy link
Contributor

@mejsullivan Yes, it's accessible! 😃

Just wanted to make a note that there's still a potential to make an accessibility enhancement that will help indicate that the ribbon is a special feature/offer/discount.

@mejsullivan
Copy link

Is that what the visual (the shape) is trying to tell people? Or does the text in the ribbon provide that content?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium medium priority item status: completed 💥 issue completed status: DPA design approved approved design by the DPA that can be picked for development type: new component 🎁 New component
Projects
Community Backlog
  
Released
Development

No branches or pull requests

8 participants