Hylozoic / hylo-evo

Hylo UI
Apache License 2.0
36 stars 10 forks source link

Evo 1299 increase title len #1583

Closed KevinTriplett closed 8 months ago

KevinTriplett commented 8 months ago

closes #1299 increase title length from 50 to 80, with tests and updates to other files to allow passing tests.

In the following screenshots, ignore the comment, the full 80 character title is shown

Screenshot 2024-02-01 at 11 54 26 AM Screenshot 2024-02-01 at 11 55 37 AM Screenshot 2024-02-01 at 11 55 47 AM Screenshot 2024-02-01 at 11 55 57 AM Screenshot 2024-02-01 at 11 56 06 AM Screenshot 2024-02-01 at 11 56 15 AM Screenshot 2024-02-01 at 11 56 27 AM Screenshot 2024-02-01 at 11 56 42 AM Screenshot 2024-02-01 at 11 56 52 AM
tibetsprague commented 8 months ago

There's a few more things/places I want to make sure you've tested

KevinTriplett commented 8 months ago
Screenshot 2024-02-05 at 9 15 02 PM Screenshot 2024-02-05 at 9 15 09 PM Screenshot 2024-02-05 at 9 15 16 PM Screenshot 2024-02-05 at 9 15 22 PM Screenshot 2024-02-05 at 9 15 54 PM Screenshot 2024-02-05 at 9 16 00 PM Screenshot 2024-02-05 at 9 16 21 PM Screenshot 2024-02-05 at 9 16 28 PM Screenshot 2024-02-05 at 9 25 12 PM Screenshot 2024-02-05 at 9 27 03 PM Screenshot 2024-02-05 at 9 27 58 PM Screenshot 2024-02-05 at 9 28 09 PM Screenshot 2024-02-05 at 9 28 55 PM Screenshot 2024-02-05 at 9 29 09 PM Screenshot 2024-02-05 at 9 29 19 PM Screenshot 2024-02-05 at 9 30 37 PM Screenshot 2024-02-05 at 9 31 00 PM
KevinTriplett commented 8 months ago

@tibetsprague , Thanks for advising testing it as an event and resource and such, with the different views. It seems well behaved, if clipping the title is what's desired / expected. Opening the item displays the full title with no overlapping. Looks good, but I wouldn't say I did an exhaustive test, more like a sample with no obvious errors.

tibetsprague commented 8 months ago

looks good except for my one comment about the error message

KevinTriplett commented 8 months ago

Test and code reverted

KevinTriplett commented 8 months ago

@tibetsprague I've scheduled a zoom meet so you can walk me through the way shallow wrappers work. I'm feeling a bit dense -- I've coded something that seems like it should work but the wrapper find method is returning an empty object despite me asking for even just a 'div' element. Reading the manuals isn't helping me for some reason. Thanks


  it('renders announcement option with admin in props', () => {
    const props = {
      ...baseProps,
      canModerate: true
    }
    const wrapper = shallow(<PostEditor {...props} />)
    expect(wrapper).toMatchSnapshot()
    expect(wrapper.find('span[data-tip="Send Announcement"]')).not.toBeNull()