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

Label dat as 'Secure P2P' in the navbar (close #279) #283

Merged
merged 2 commits into from
Feb 7, 2017
Merged

Conversation

pfrazee
Copy link
Member

@pfrazee pfrazee commented Feb 7, 2017

Per @pmario's suggestion in #278, and per the decisions in #279, this PR updates the navbar UI as follows:

screen shot 2017-02-06 at 10 06 41 pm

The "Learn more" link goes to https://github.com/beakerbrowser/beaker/wiki/Is-Dat-%22Secure-P2P%3F%22

@pfrazee pfrazee merged commit 1d9a23d into master Feb 7, 2017
@pfrazee pfrazee deleted the dat-desc-in-ui branch February 7, 2017 04:08
@pmario
Copy link
Contributor

pmario commented Feb 8, 2017

Strange. .. I did click the label with latest master build. ... Nothing happens :/

@pfrazee
Copy link
Member Author

pfrazee commented Feb 8, 2017

That is strange. If you open the Beaker dev tools (under BeakerDev) do you see an error in the console?

@pmario
Copy link
Contributor

pmario commented Feb 8, 2017

nope. Did already check the DOM. the div and all the elements are there. .. They are just invisible. Seems to be related to #285

@pfrazee
Copy link
Member Author

pfrazee commented Feb 8, 2017

Seems to be related to #285

What electron version are you on? I haven't merged that PR yet so you should be on 1.5.0

@pmario
Copy link
Contributor

pmario commented Feb 8, 2017

  • you are right. it's 1.5.0
  • I did checkout commit fb1436c to a new branch
    • It works there
  • now I'll apply the changes and see what happens.

I'll post here.

@pmario
Copy link
Contributor

pmario commented Feb 8, 2017

found it: 72d8c12#commitcomment-20801743

@pfrazee
Copy link
Member Author

pfrazee commented Feb 8, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants