Merge lp:~artem-anufrij/webby-browser/webby-browser into lp:webby-browser
Proposed by
Artem Anufrij
on 2015-03-23
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Erasmo Marín on 2015-03-24 | ||||
Approved revision: | 12 | ||||
Merged at revision: | 10 | ||||
Proposed branch: | lp:~artem-anufrij/webby-browser/webby-browser | ||||
Merge into: | lp:webby-browser | ||||
Diff against target: |
100 lines (+34/-10) 3 files modified
CMakeLists.txt (+6/-5) src/ApplicationsView.vala (+27/-4) src/Webby.vala (+1/-1) |
||||
To merge this branch: | bzr merge lp:~artem-anufrij/webby-browser/webby-browser | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Erasmo Marín | 2015-03-23 | Pending | |
Review via email:
|
To post a comment you must log in.
Artem Anufrij (artem-anufrij) wrote : | # |
Erasmo Marín (erasmo-marin) wrote : | # |
Thanks, it looks good. The only thing you need to fix is line 74, can you put the "}" with the "else" in the same line? that's all.
lp:~artem-anufrij/webby-browser/webby-browser
updated
on 2015-03-23
- 11. By Artem Anufrij on 2015-03-23
-
code style
- 12. By Artem Anufrij on 2015-03-23
-
code style
Artem Anufrij (artem-anufrij) wrote : | # |
done :)
Screenshot:
https:/ /bugs.launchpad .net/webby- browser/ +bug/1429613/ +attachment/ 4353631/ +files/ %D0%A1% D0%BD%D0% B8%D0%BC% D0%BE%D0% BA%20%D1% 8D%D0%BA% D1%80%D0% B0%D0%BD% D0%B0%20% D0%BE%D1% 82%202015- 03-23%2019% 3A03%3A17. png