Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Adding custom bookmark name text box not clearing when deleted #7691

Closed
TheNightRider12 opened this issue Mar 14, 2017 · 8 comments
Closed

Comments

@TheNightRider12
Copy link

TheNightRider12 commented Mar 14, 2017

Test plan

  1. automated bookmark tests should pass
  2. try editing a bookmark and fully deleting the title. the title field should be empty.

Original issue description

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    When trying to clear the text box in "Bookmark Page" so I can set a custom name the original website's title keeps appearing in the text box after deleting it. As soon as I delete the first letter in the name, the original website title pops back into the box.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10

  • Brave Version (revision SHA):
    1db81cb

  • Steps to reproduce:

    1. Go to a website
    2. click on "Bookmark page"
    3. when the box appears try and rename the name of the bookmark
    4. As soon as you delete the first letter the original text appears again.
  • Actual result:
    The name of the website keeps popping back up and I can't set a custom name for it

  • Expected result:
    To be able to name the bookmark what I want it to say without having the original text appear again after I delete the first letter. I think it would be better to be ab;e to have the text box completely blank when you want to name a bookmark. Yes still have the original webpage be there at first but after everything is gone, the original text shouldn't reappear.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes they can.

  • Is this an issue in the currently released version?
    0.13.5 in "Releases"

  • Can this issue be consistently reproduced?
    Yes it can

  • Extra QA steps:

    1. Go to a website
    2. Go to "Bookmarks"
    3. Press "Bomark page"
    4. Try to rename the website to a custom name for the bookmark
    5. When you try to delete all of the text that is already in the text box, the original text will reappear after you delete the first letter
  • Screenshot if needed:
    Gif of what I see

  • Any related issues:

    • None that I can see
@cndouglas
Copy link

Confirmed with 0.13.5. Deleting each character individually appears to work, but deleting the entire text does not work.

@cndouglas cndouglas added the bug label Mar 14, 2017
@cndouglas
Copy link

This problem is caused by 57108ea. @diracdeltas

cndouglas referenced this issue Mar 14, 2017
Fix #7291

Test plan:
1. open pyramid 0, enable sync, bookmark something.
2. open pyramid 1, sync to pyramid 0, wait for the synced bookmark to appear.
3. click 'edit bookmark...' on the synced bookmark in pyramid 1. the title field should not be empty.
@diracdeltas diracdeltas self-assigned this Mar 14, 2017
@diracdeltas diracdeltas added this to the 0.14.0 milestone Mar 14, 2017
@alexwykoff alexwykoff mentioned this issue Mar 14, 2017
44 tasks
diracdeltas added a commit that referenced this issue Mar 14, 2017
Fix #7691

Test Plan:
1. automated bookmark tests should pass
2. try editing a bookmark and fully deleting the title. the title field should be empty.
@luixxiul
Copy link
Contributor

I cannot confirm this was fixed on Windows 10 32 bit

@TheNightRider12
Copy link
Author

It's been fixed for me. 64bit

@luixxiul
Copy link
Contributor

Maybe am I missing something?

bookmark

@TheNightRider12
Copy link
Author

The bug I made was when you would delete everything completely, the original text would reappear and it wouldn't go away.

@luixxiul
Copy link
Contributor

Ah I read just too fast, sorry.

The input field becomes empty as the gif above shows.

@diracdeltas
Copy link
Member

@luixxiul yep, that is the expected behavior

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.